-
Notifications
You must be signed in to change notification settings - Fork 0
feat(eql_v2)!: collapse ste_vec ORE terms to single oc field #219
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
Merged
Merged
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
c1b2c3e
feat(eql_v2)!: collapse ste_vec ORE terms to single oc field
coderdan 069697c
fix(ci): allowlist `eql_v2.ore_cllw` and `has_ore_cllw` in splinter
coderdan 1be0646
feat(eql_v2)!: strict separation between root payload and sv-element …
coderdan 70d803d
test(index_compare): migrate ore_cllw tests to typed entry API
coderdan 3fab115
test(operator_compare): migrate to entry shape + new error wording
coderdan 2e57831
refactor(hash): assume `hm` at root, drop coalesce fallback
coderdan 11cb7b6
perf(ore_cllw): remove early exit + use bitmask in compare_term_bytes
coderdan e32999b
test: add `build_synthetic_ste_vec(id)` helper for v2.3-shape ste_vec…
coderdan a77aeef
test: rewrite tests/ste_vec.sql to v2.3-shape inline INSERTs
coderdan 4872c10
feat(ste_vec_entry)!: tighten DOMAIN check to require s + c + hm
coderdan 6bde3e3
test: re-enable ignored ste_vec tests + add `hm` to legacy fixtures
coderdan 7b6ed5f
fix(ste_vec_entry)!: DOMAIN check is XOR(hm, oc), not AND(hm, c)
coderdan 06d706a
chore(v2.3): codify XOR(hm, oc) contract end-to-end
coderdan ecb7ee6
docs(v2.3): address U-006 review feedback
coderdan 2558bca
docs(changelog): add PR links to unlinked v2.3 entries
coderdan 7a78645
fix(src): address Dan + CodeRabbit source-side review feedback
coderdan 787074d
fix(fixtures): drop root c from ste_vec_vast + b3 from hm-backstop
coderdan 5ea4a4f
test: address Dan's test-side review feedback
coderdan File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.