Skip to content

test: add unittest#231

Merged
lxy-9602 merged 8 commits into
alibaba:mainfrom
lxy-9602:add-unit-test
Apr 17, 2026
Merged

test: add unittest#231
lxy-9602 merged 8 commits into
alibaba:mainfrom
lxy-9602:add-unit-test

Conversation

@lxy-9602
Copy link
Copy Markdown
Collaborator

@lxy-9602 lxy-9602 commented Apr 16, 2026

Purpose

Add unittests.
Remove partial dependencies from common to core, to eliminate cyclic dependency risks.
int->int32_t

No Linked issue.

Tests

BitmapFileIndexMetaTest, TestSmallBlockSizeForcesMultipleBlocks
ChunkedDictionaryTest, TestChunkFindFirstKeyFastPath
RangeBitmapFileIndexTest, TestRangeBitmapEdgeCases
BitmapGlobalIndexTest, TestEmpty
CacheInputStreamTest
MemorySegmentTest, TestEqual
MemorySliceTest
LookupMergeTreeCompactRewriterTest, TestMultipleCompression

API and Format

Documentation

Generative AI tooling

Partially Generated-by: Claude-4.6-Opus

lszskye
lszskye previously approved these changes Apr 17, 2026
lszskye
lszskye previously approved these changes Apr 17, 2026
@lxy-9602
Copy link
Copy Markdown
Collaborator Author

Given that http://archive.ubuntu.com is down (breaking CI) and main has existing regressions, we'll force-merge this PR and skip the GCC 8 job for now.

Action required later: re-enable the test once infrastructure recovers.

@lxy-9602 lxy-9602 merged commit 393b613 into alibaba:main Apr 17, 2026
8 of 13 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.

3 participants