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.

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

Jump to bottom

Document limits: only apply Max Object Properties limits on indexed fields #821

Closed
tatu-at-datastax opened this issue Jan 22, 2024 · 0 comments · Fixed by #844
Closed

Document limits: only apply Max Object Properties limits on indexed fields #821

tatu-at-datastax opened this issue Jan 22, 2024 · 0 comments · Fixed by #844
Assignees
@tatu-at-datastax

Comments

@tatu-at-datastax
Copy link
Contributor

tatu-at-datastax commented Jan 22, 2024 •
edited
Loading

As part of evaluation of Document limits (see DocumentLimitsConfig.java / CONFIGURATION.md), we want to:

  • Change validation of "max-object-properties" and "max-document-properties" to only cover indexed fields (that is, exclude no-index fields). This can be done by moving validation logic to occur after pruning no-index fields.
@tatu-at-datastax tatu-at-datastax self-assigned this Jan 25, 2024
tatu-at-datastax added a commit that referenced this issue Jan 29, 2024
@tatu-at-datastax
Fixes #821: apply max-object-properties/max-doc-properties only on in…
01203aa 
…dexable Objects
@tatu-at-datastax tatu-at-datastax mentioned this issue Jan 29, 2024
Merged
4 tasks
maheshrajamani added a commit that referenced this issue Jan 31, 2024
@tatu-at-datastax @maheshrajamani
Fixes #821: apply max-object-properties/max-doc-properties only on in…
9ae532d 
…dexed Objects (#844)

Co-authored-by: Mahesh Rajamani <99678631+maheshrajamani@users.noreply.github.com>
@tatu-at-datastax tatu-at-datastax changed the title Document limits: only apply Max Object/Document Properties limits on indexed fields Document limits: only apply Max Object Properties limits on indexed fields Jan 31, 2024
@tatu-at-datastax tatu-at-datastax mentioned this issue Jan 31, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tatu-at-datastax tatu-at-datastax

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes #821: apply max-object-properties/max-doc-properties only on indexed Objects stargate/data-api
1 participant
@tatu-at-datastax