Skip to content

OAK-12171 Cost estimation for is (not) null checks are wrong #2825

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
thomasmueller merged 7 commits into from
Apr 2, 2026

OAK-12171 Cost estimation for is (not) null checks are wrong

8ff34e5
Select commit
Loading
Failed to load commit list.
Merged

OAK-12171 Cost estimation for is (not) null checks are wrong #2825

OAK-12171 Cost estimation for is (not) null checks are wrong
8ff34e5
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / SonarCloud Code Analysis succeeded Apr 2, 2026 in 2m 38s

Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
97.1% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check warning on line 183 in oak-search-elastic/src/test/java/org/apache/jackrabbit/oak/plugins/index/elastic/ElasticIndexPlannerCommonTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this use of "Thread.sleep()". 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AZ1ODKKfG7m089GVINv1&open=AZ1ODKKfG7m089GVINv1&pullRequest=2825

Check warning on line 30 in oak-search/src/test/java/org/apache/jackrabbit/oak/plugins/index/IndexPlannerCommonTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused import 'org.apache.jackrabbit.oak.plugins.index.search.FieldNames'. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AZ1ODKFOG7m089GVINv0&open=AZ1ODKFOG7m089GVINv0&pullRequest=2825

Check warning on line 1549 in oak-search/src/test/java/org/apache/jackrabbit/oak/plugins/index/IndexPlannerCommonTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this unused method parameter "async". 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AZ1ODKFOG7m089GVINvz&open=AZ1ODKFOG7m089GVINvz&pullRequest=2825

Check warning on line 183 in oak-search-elastic/src/test/java/org/apache/jackrabbit/oak/plugins/index/elastic/ElasticIndexPlannerCommonTest.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

Remove this block of code, fill it in, or add a comment explaining why it is empty. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AZ1ODKKfG7m089GVINv2&open=AZ1ODKKfG7m089GVINv2&pullRequest=2825

Check warning on line 862 in oak-search/src/main/java/org/apache/jackrabbit/oak/plugins/index/search/spi/query/FulltextIndexPlanner.java

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / SonarCloud Code Analysis

A "Brain Method" was detected. Refactor it to reduce at least one of the following metrics: LOC from 67 to 64, Complexity from 22 to 14, Nesting Level from 5 to 2, Number of Variables from 14 to 6. 

See more on https://sonarcloud.io/project/issues?id=org.apache.jackrabbit%3Ajackrabbit-oak&issues=AZ1Eo3UOxuRvoptrDwMY&open=AZ1Eo3UOxuRvoptrDwMY&pullRequest=2825
View more details on SonarQubeCloud