Skip to content
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

Exits prematurely if Num Of Values per segment column crosses threshold #2

Draft
wants to merge 48 commits into
base: master
Choose a base branch
from

Conversation

Harnoor7
Copy link
Owner

@Harnoor7 Harnoor7 commented Nov 18, 2024

Problem: Num of values for a multi valued column exceeds Integer.MAX_VALUE and overflows resulting in RuntimeException during indexing.

Proposed Solution: Stop segment consumption if num of values for any multi valued column crosses a defined threshold.

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. release-notes (**)
  2. Remove these instructions before publishing the PR.

(*) Other labels to consider:

  • testing
  • dependencies
  • docker
  • kubernetes
  • observability
  • security
  • code-style
  • extension-point
  • refactor
  • cleanup

(**) Use release-notes label for scenarios like:

  • New configuration options
  • Deprecation of configurations
  • Signature changes to public methods/interfaces
  • New plugins added or old plugins removed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant