-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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.
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
Add Helper Functions in StarTreeBuilderUtils and StarTreeV2BuilderConfig #12361
Add Helper Functions in StarTreeBuilderUtils and StarTreeV2BuilderConfig #12361
Conversation
* Added helper function to compare 2 StarTree builder configs. * Added unit test for generateDefaultConfig from JsonNode segment metadata.
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## master #12361 +/- ##
============================================
- Coverage 61.72% 61.68% -0.04%
Complexity 207 207
============================================
Files 2426 2426
Lines 132667 132776 +109
Branches 20499 20541 +42
============================================
+ Hits 81888 81906 +18
- Misses 44774 44859 +85
- Partials 6005 6011 +6
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM otherwise.
...egment-local/src/main/java/org/apache/pinot/segment/local/startree/StarTreeBuilderUtils.java
Show resolved
Hide resolved
...egment-local/src/main/java/org/apache/pinot/segment/local/startree/StarTreeBuilderUtils.java
Show resolved
Hide resolved
…fig (apache#12361) * changed some stuffs * fixed an issue for generating default builder configs * optimize passing segment metadata * Add helper function and unit test * Added helper function to compare 2 StarTree builder configs. * Added unit test for generateDefaultConfig from JsonNode segment metadata. * add additional unit tests. * addressing comments
Added the following helper functions: