Skip to content

Conversation

@luoluoyuyu
Copy link
Member

As the title said

@luoluoyuyu luoluoyuyu marked this pull request as draft November 4, 2024 15:40
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hi, this is your first pull request in the Apache TsFile project. Thanks for your contribution! TsFile will be better because of you.

@luoluoyuyu luoluoyuyu marked this pull request as ready for review November 5, 2024 16:05
Copy link
Contributor

@jt2594838 jt2594838 left a comment

Choose a reason for hiding this comment

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

LGTM, but I am not sure if it should be skipped or a place holder is needed.

@jt2594838 jt2594838 merged commit 975210c into apache:develop Nov 6, 2024
jt2594838 pushed a commit that referenced this pull request Nov 11, 2024
* Fix the problem that ChunkMeta of empty Chunk is not processed

* add TsFileReaderEmptyChunkTest

* fix

* add licensed

* modify code format
jt2594838 pushed a commit that referenced this pull request Nov 11, 2024
* Fix the problem that ChunkMeta of empty Chunk is not processed

* add TsFileReaderEmptyChunkTest

* fix

* add licensed

* modify code format

(cherry picked from commit 975210c)
jt2594838 pushed a commit that referenced this pull request Nov 11, 2024
* Fix the problem that ChunkMeta of empty Chunk is not processed

* add TsFileReaderEmptyChunkTest

* fix

* add licensed

* modify code format

(cherry picked from commit 975210c)
(cherry picked from commit 3108a49)
jt2594838 pushed a commit that referenced this pull request Nov 11, 2024
* Fix the problem that ChunkMeta of empty Chunk is not processed

* add TsFileReaderEmptyChunkTest

* fix

* add licensed

* modify code format

(cherry picked from commit 8310640)
@luoluoyuyu luoluoyuyu deleted the fix-reader branch November 18, 2025 08:47
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