Add rerouting error feedback refreshing logic#358
Merged
WeiXinChan merged 6 commits intohbase_441from Aug 13, 2025
Merged
Conversation
JackShi148
force-pushed
the
fix_rerouting_err_feedback
branch
5 times, most recently
from
c23dca0 to
312d118
Compare
JackShi148
force-pushed
the
fix_rerouting_err_feedback
branch
from
312d118 to
e47a41f
Compare
JackShi148
force-pushed
the
fix_rerouting_err_feedback
branch
from
e47a41f to
a1b39d6
Compare
JackShi148
force-pushed
the
fix_rerouting_err_feedback
branch
from
a1b39d6 to
aeea4c4
Compare
Contributor
|
LGTM |
WeiXinChan
added a commit
that referenced
this pull request
Sep 1, 2025
* Hbase put perf (#368) * new hbase protocal (#363) * new hbase request and result for multi-cf * odp support new hbase protocol * order table_name when get by tablegroup --------- Co-authored-by: vanson <43193589+WeiXinChan@users.noreply.github.com> * Add rerouting error feedback refreshing logic (#358) * refresh table meta and locatino based on flag, refresh cache after retry successfully in server * remove lsop + OB_NOT_MASTER * change debug log to warn * make execute and operation time longer * add debug log * revert log level and operation timeout * Fix old client and odp disable to refresh tableEntry if using new server (#371) * add server_can_retry flag to enable retry capacity for new client operations * add ObBinlogRowImageType for QueryAndMutateRequest to skip the space of encode * heap table tests (#374) * Fix scan sess missing transfer and single cf table put (#375) * generate new query request when hash_not_exist during query_next * fix isKeyInRange when encounter MinObObj or MaxObObj * fix hbase put sinlge cf table * do not refresh tableEntry if require_rerouting is true in odp mode * add valid status check when add new obtable (#377) * remove invalid table (#378) * set dirty and add ip to suspect list (#379) --------- Co-authored-by: vanson <43193589+WeiXinChan@users.noreply.github.com>
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
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
Add rerouting flag and refresh meta flag to trigger refreshing when retry successfully in server encountering rerouting errors.
Solution Description