Skip to content

Conversation

@19-hanhan
Copy link
Contributor

What problem does this PR solve?

  1. Add commit_ts to TableMeeta to record transaction status, and store its commit_ts in the associated Meta of TableMeeta.
  2. kv_instance obtains visible block_ids via GetTableSegmentBlocks, where commit_ts is added to identify the block_id for the current transaction.
  3. new_txn acquires an iterator for block data through NewTxnGetVisibleRangeState, add commit_ts_ to fetch the block data for the current transaction.

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • Test cases
  • Python SDK impacted, Need to update PyPI

19-hanhan added 7 commits May 28, 2025 15:38
Signed-off-by: Long <2262328655@qq.com>
Signed-off-by: Long <2262328655@qq.com>
Signed-off-by: Long <2262328655@qq.com>
Signed-off-by: Long <2262328655@qq.com>
Signed-off-by: Long <2262328655@qq.com>
@19-hanhan 19-hanhan added the ci PR can be test label May 29, 2025
Signed-off-by: Long <2262328655@qq.com>
@JinHai-CN JinHai-CN removed the ci PR can be test label May 29, 2025
19-hanhan added 4 commits May 30, 2025 15:57
Signed-off-by: Long <2262328655@qq.com>
…eter, used to query cases where `commit_ts` is not present in `SegmentMeta`.

Signed-off-by: Long <2262328655@qq.com>
Signed-off-by: Long <2262328655@qq.com>
@19-hanhan 19-hanhan added ci PR can be test and removed ci PR can be test labels May 30, 2025
@JinHai-CN JinHai-CN merged commit 8a080fa into infiniflow:main Jun 3, 2025
3 checks passed
yuzhichang pushed a commit that referenced this pull request Jun 3, 2025
### What problem does this PR solve?

1. Add commit_ts to TableMeeta to record transaction status, and store
its commit_ts in the associated Meta of TableMeeta.
2. kv_instance obtains visible block_ids via GetTableSegmentBlocks,
where commit_ts is added to identify the block_id for the current
transaction.
3. new_txn acquires an iterator for block data through
NewTxnGetVisibleRangeState, add commit_ts_ to fetch the block data for
the current transaction.

### Type of change

- [x] Bug Fix (non-breaking change which fixes an issue)
- [x] Test cases
- [x] Python SDK impacted, Need to update PyPI

---------

Signed-off-by: Long <2262328655@qq.com>
@19-hanhan 19-hanhan deleted the dev branch June 3, 2025 06:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci PR can be test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants