Fix global route refresh wrong #231
Merged
Conversation
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
shenyunlong
approved these changes
Nov 14, 2024
Contributor
|
LGTM |
maochongxin
pushed a commit
that referenced
this pull request
Dec 4, 2024
* fix global index route refresh wrong when route need refresh * fix global index route wrong * revert unused commit
WeiXinChan
added a commit
that referenced
this pull request
Dec 6, 2024
…gle-partition operation (#178) * git reset to no test util commit * add ObFetchPartitionMetaType in request, pass re-fetch meta * pass all situations to get ODP partition meta * fix LSBatch in HBase mode, pass getPartition self-testing * correct format * revert LSBatch negateHbaseTimestamp * revert batch atomic change * correct substr test * compatible with table master branch * update addScanRange test with partition * update odp rpc lock timeout, add odp range query * add odp refresh lock timeout into initProperties * remove getPartition with only row key values * delete useless getStart and getEnd interface in Partition * format code * remove testing output * bugfix hbase mode using odp mode does not add row key; format code * add exception dealing in fetching odp partition meta * correct test case * do not calculate partition_id in ODP mode when using index * do not calculate partition_id in ODP mode when using index query * fetch the latest partition information in ocp mode everytime using getPartition * add refresh flag in getPartition to get latest table and partition information * Optimize SQL for refreshing table location information * partical refresh * Fix infinite loop caused by removed reference * fix lsop retry fail * Remove unnecessary comments and format code * Fix frequent refresh lock failures due to short refresh interval * Fix frequent refresh lock failures due to short refresh interval * add result code -4723 * correct test; getPartition with rowKey do not need to add rowKey before * correct test cases * fix batch in odp mode after merge * bugfix query with part_id * fix review * add -4138 * fix review: add ut for byteutil * fix lsop refresh location * fix: correct reverse-scan results affected by retry logic * set ODP mode retry number to 3 * revert retry time * feature: Optimize partial refresh and add retry for executeMutation/query (#213) * remove lock in refreshTableLocationByTabletId * fix refresh sql * fix refresh interval * add retry logic for common query * add retry logic for executeMutation * add retry logic for executeMutation * add retry logic for ObTableQueryAndMutateRequest * fix: correct ineffective retry logic * retry logic for ObTableClient execute * fix * fix * opt for errorMsg when execute batch in the scene of table not exist * fix ls batch errMsg * update pom.xml * remove some unused dependencies in pom.xml * fix 3.x null exception (#214) * fix * fix regress * [fix] add threshold for refresh table entry with location (#220) * add threshold for refresh table entry with location * [Fix] remove TABLE_ENTRY_LOCATION_REFRESH_THRESHOLD in Property * [Fix] fix compile error --------- Co-authored-by: shenyunlong.syl <shenyunlong.syl@oceanbase.com> * Fix the issue where the tablet_id is incorrect when the partition table has a single partition (#226) * [fix] 3.x compatible (#223) * fix * fix * [fix] test case stuck in await (#227) * fix * fix * refine * partition=1 error fix * fix after merge and correct some test cases * fix getPartition after merge * add logic id into partition * fix global index route refresh wrong when route need refresh (#228) * Fix global route refresh wrong (#231) * fix global index route refresh wrong when route need refresh * fix global index route wrong * revert unused commit * revert ls batch, do not fetch odp partition meta in ls batch * [Chore] remove the use of sofa common log * modify version control for direct-load partition (#236) * param (#221) * [fix] multi-cf retry table not exist (#229) * fix global index route wrong when need partial route refresh (#232) * fix global index route wrong when need partial route refresh * Ensure PartitionInfo is Exposed Only After Leader is Found During Single Shard Refresh --------- Co-authored-by: maochongxin <maochongxin@gmail.com> * feature: hbase compatible 2.x (#242) * support hbase scan renewLease (#211) * hbase tablename bug fix (#237) * hbase empty family scan error (#239) * hbase tablename bug fix * hbase empty family scan error --------- Co-authored-by: stuBirdFly <84010733+stuBirdFly@users.noreply.github.com> * everytime force new if flag is true * add comments * fix -5150 error and try to refresh route after relogin * fix getAllPartitionInternal * fix -5150 tenent not exists * remove useless import * revert global index test case --------- Co-authored-by: maochongxin <maochongxin@gmail.com> Co-authored-by: GroundWu <1175416256@qq.com> Co-authored-by: WeiXinChan <chenwx6728@163.com> Co-authored-by: miyuan-ljr <884244693@qq.com> Co-authored-by: shenyunlong.syl <shenyunlong.syl@oceanbase.com> Co-authored-by: stuBirdFly <84010733+stuBirdFly@users.noreply.github.com> Co-authored-by: medcll <53672709+medcll@users.noreply.github.com>
maochongxin
pushed a commit
that referenced
this pull request
Dec 10, 2024
* fix global index route refresh wrong when route need refresh * fix global index route wrong * revert unused commit
maochongxin
pushed a commit
that referenced
this pull request
Jan 3, 2025
* fix global index route refresh wrong when route need refresh * fix global index route wrong * revert unused commit
shenyunlong
added a commit
that referenced
this pull request
Jan 3, 2025
* Optimize SQL for refreshing table location information * partical refresh * Fix infinite loop caused by removed reference * fix lsop retry fail * Remove unnecessary comments and format code * Fix frequent refresh lock failures due to short refresh interval * Fix frequent refresh lock failures due to short refresh interval * add result code -4723 * fix review * add -4138 * fix review: add ut for byteutil * fix lsop refresh location * Merge pull request #204 from oceanbase/retry_batchops_merge_master Enhance Client Support for Partition Splitting * feature: Optimize partial refresh and add retry for executeMutation/query (#213) * remove lock in refreshTableLocationByTabletId * fix refresh sql * fix refresh interval * add retry logic for common query * add retry logic for executeMutation * add retry logic for executeMutation * add retry logic for ObTableQueryAndMutateRequest * fix: correct ineffective retry logic * retry logic for ObTableClient execute * fix * fix * fix 3.x null exception (#214) * fix * fix regress * [fix] add threshold for refresh table entry with location (#220) * add threshold for refresh table entry with location * [Fix] remove TABLE_ENTRY_LOCATION_REFRESH_THRESHOLD in Property * [Fix] fix compile error --------- Co-authored-by: shenyunlong.syl <shenyunlong.syl@oceanbase.com> * [fix] 3.x compatible (#223) * fix * fix * [fix] test case stuck in await (#227) * fix * fix * refine * Fix global route refresh wrong (#231) * fix global index route refresh wrong when route need refresh * fix global index route wrong * revert unused commit * fix global index route wrong when need partial route refresh (#232) * fix global index route wrong when need partial route refresh * Ensure PartitionInfo is Exposed Only After Leader is Found During Single Shard Refresh --------- Co-authored-by: maochongxin <maochongxin@gmail.com> * fix PartitionNumOneTest routed to the wrong server * add error code 5627 * fix extendType parse object to comparable * fix refresh location param * fix log; add retry interval for executeWithRetries * Add quick path: return quickly if recently refreshed * add log for batchOpsImpl * fix * fix table.entry.refresh.interval.wait * fix global index route error * fix * TABLE_ENTRY_LOCATION_REFRESH_THRESHOLD -> 0 * add retry code =4242 * fix getPartitionLocation nullptr && resolve conflict error * fix resolve conflict error * fix getLs nullptr * fix dr nullptr * fix global index nullptr * fix unexpected addr expired * fix unexpected addr expired * Reduce log printing and optimize log output content; * fix update lsId if location exists * fix log * fix lsop -5200 with table_group * fix rpc.execute.timeout * fix log * fix log && fix getPartitionReplica return wrong part id --------- Co-authored-by: miyuan-ljr <884244693@qq.com> Co-authored-by: shenyunlong.syl <shenyunlong.syl@oceanbase.com> Co-authored-by: GroundWu <1175416256@qq.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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
Solution Description