release: v0.6.6-dev — TCK 383/402, type inference ANY fix, chdb resource caps#270
Merged
release: v0.6.6-dev — TCK 383/402, type inference ANY fix, chdb resource caps#270
Conversation
…caps - type_inference.rs: switch property-filter from ALL→ANY semantics so OR predicates across multi-label nodes keep all candidate types; exclude IS NULL / IS NOT NULL accesses from candidate pruning (missing props → NULL, not an error) — fixes MatchWhere1 and Null1 TCK failures - clickgraph-embedded: add max_memory_usage_bytes to SystemConfig; actually apply max_threads and max_memory_usage via SET commands at session init - clickgraph-tck: cap TCK session to 4 threads / 4 GiB; remove stale mutations_sync=2 SET; bump chdb-rust 1.3.0 → 1.3.1 - README.md: move motivation to docs/motivation.md, add modes summary, fix skills link to absolute GitHub URL, mention cg CLI in tagline - clickgraph-tck/README.md: update to 383/402 (95.3%), 0 failures Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- type_inference.rs: proper IS NULL / IS NOT NULL handling — thread a null_check map through extract_props_from_expr/plan; IS NULL/IS NOT NULL operands go to null_check (not regular). Pruning uses regular ∪ null_check with ANY semantics, but skips pruning entirely when only null-check accesses exist (missing properties return NULL, not an error). Fixes the unsound early return that could drop types in `WHERE n.p IS NOT NULL OR n.q = 5`. - database.rs: revert #[non_exhaustive] (prohibits struct literal construction in external crates even with ..default()); document the All-Option convention as the forward-compat strategy instead. - README.md: use relative links (docs/motivation.md, skills/README.md) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
WHERE n.p1 = 12 OR n.p2 = 13across multi-label nodes works correctly: property filtering uses ANY semantics. IS NULL / IS NOT NULL properties tracked in a separatenull_checkmap — used for ANY-semantics pruning alongside regular accesses, but pruning is skipped entirely when only null-check accesses exist (missing properties return NULL, not an error). Fixes MatchWhere1 and Null1 TCK failures, and correctly handlesWHERE n.p IS NOT NULL OR n.q = 5disjunctions.SystemConfiggainsmax_memory_usage_bytes; bothmax_threadsandmax_memory_usageare now applied viaSETat session init (previouslymax_threadswas documented but never sent to chdb)docs/motivation.md, modes summary added,cgCLI mentioned in tagline, relative links used throughoutTest plan
cargo test -p clickgraph-tck— 383 passed, 19 skipped, 0 failedcargo fmt --all/cargo clippy --all-targetsclean🤖 Generated with Claude Code