Skip to content

test: cover global indexes, nested projections and changelog scans - #511

Merged
lxy-9602 merged 1 commit into
apache:mainfrom
lszskye:add_ut_release_0.4
Nov 4, 2026
Merged

lxy-9602 merged 1 commit into
apache:mainfrom
lszskye:add_ut_release_0.4

Conversation

@lszskye

@lszskye lszskye commented Nov 4, 2026

Copy link
Copy Markdown
Collaborator

Purpose

Improve test coverage for:

  • Recursive decoding of ORC dictionary arrays inside Struct and Map fields during global index construction.
  • StartsWith, EndsWith, Contains and Like predicates in global index scans.
  • GlobalIndexScan creation with partition predicates and default/custom memory pools.
  • Nested Struct and Map-key projection detection, including Variant access.
  • Recursive List/Map pruning with added fields and Variant projections.
  • Value-statistics filtering in CHANGELOG mode for lookup, full-compaction and input producers.

Tests

  • GlobalIndexTest.TestWriteIndexDecodesNestedOrcDictionaries
  • GlobalIndexTest.TestScanIndexWithPartitionPredicate
  • NestedProjectionUtilsTest.PruneDataTypeNestedListMapWithAddedFieldAndVariantAccess
  • NestedProjectionUtilsTest.HasNestedSubfieldProjectionVariantAccessReturnsFalse
  • NestedProjectionUtilsTest.HasNestedSubfieldProjectionNestedStruct
  • NestedProjectionUtilsTest.HasNestedSubfieldProjectionMapKey
  • KeyValueFileStoreScanTest.TestChangelogValueFilterByProducer
  • GlobalIndexTest.TestScanIndex with four string predicate checks.

API and Format

No changes to public APIs, storage formats or protocols.

Documentation

No documentation changes are needed. This PR only adds tests.

Generative AI tooling

Generated-by: OpenAI Codex (GPT-6)

@lxy-9602 lxy-9602 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@lxy-9602
lxy-9602 merged commit ab0f5ba into apache:main Nov 4, 2026
16 checks passed
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.

2 participants