Re: [PR] Fixes int overflow for num values [pinot]

2025-01-15 Thread via GitHub
noob-se7en commented on code in PR #14423: URL: https://github.com/apache/pinot/pull/14423#discussion_r1917467762 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/io/writer/impl/FixedBitMVForwardIndexWriter.java: ## @@ -74,7 +74,7 @@ public class FixedBitMVFor

Re: [PR] Fixes int overflow for num values [pinot]

2025-01-15 Thread via GitHub
noob-se7en commented on code in PR #14423: URL: https://github.com/apache/pinot/pull/14423#discussion_r1917467762 ## pinot-segment-local/src/main/java/org/apache/pinot/segment/local/io/writer/impl/FixedBitMVForwardIndexWriter.java: ## @@ -74,7 +74,7 @@ public class FixedBitMVFor

Re: [PR] Fixes int overflow for num values [pinot]

2024-11-11 Thread via GitHub
Harnoor7 closed pull request #14423: Fixes int overflow for num values URL: https://github.com/apache/pinot/pull/14423 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubs

Re: [PR] Fixes int overflow for num values [pinot]

2024-11-11 Thread via GitHub
Harnoor7 commented on PR #14423: URL: https://github.com/apache/pinot/pull/14423#issuecomment-2468014321 Pls ignore this PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment.

[PR] Fixes int overflow for num values [pinot]

2024-11-11 Thread via GitHub
Harnoor7 opened a new pull request, #14423: URL: https://github.com/apache/pinot/pull/14423 Instructions: 1. The PR has to be tagged with at least one of the following labels (*): 1. `feature` 2. `bugfix` 3. `performance` 4. `ui` 5. `backward-incompat` 6