Skip to content

Conversation

@JackShi148
Copy link
Contributor

Summary

Throw the original exception when initRoster failed and the cause is the same. The error code will be carried by the exception.

Solution Description

@JackShi148 JackShi148 force-pushed the fix_nonexists_namespace branch from 44fdc77 to ec857e6 Compare June 16, 2025 03:02
@WeiXinChan
Copy link
Contributor

LGTM

@WeiXinChan WeiXinChan merged commit 462531e into hbase_compat_3 Jun 16, 2025
3 checks passed
WeiXinChan added a commit that referenced this pull request Jun 25, 2025
…se Table Operations (#353)

* MetaExecutor interface definition and demo (#337)

* lsop encode opt (#339)

* opt for lsop getPayLoadContentSize

* add performa test for ObTableLsOperationRequest encode

* use ObByteBuf to encode lsop

* lsop request encode perf opt

* add default bytes cache for obkvparam and ohtablefilter

* Revert "add default bytes cache for obkvparam and ohtablefilter"

This reverts commit 8a555ef.

* add default bytes cache for obkvparam and ohtablefilter

* fix bug for indexName encode

* adapt for disable/enable table (#340)

* support hot only (#341)

Co-authored-by: WeiXinChan <chenwx6728@163.com>

* Fix TableQuery cannot refresh tablet location (#345)

* add debug log

* refresh tablet location for atomic query

* throw the same type original Exception when login (#346)

* update netty to 4.1.118.Final to avoid security problem (#349)

* Replace Fastjson with Jackson (#347)

* change fastjson to jackson

* use jackson 2.14.2, compile pass

* succeed to print log

* update jackson and maven-shade-plugin to the lateset version

* revert OcpResponseData constructors

* diable reduce pom dependencies

* [OBKV] add hbase admin ddl errcode

---------

Co-authored-by: GroundWu <1175416256@qq.com>
Co-authored-by: Shen Yunlong <44497386+shenyunlong@users.noreply.github.com>
Co-authored-by: WeiXinChan <chenwx6728@163.com>
Co-authored-by: Ziyu Shi <57038180+JackShi148@users.noreply.github.com>
Co-authored-by: shenyunlong.syl <shenyunlong.syl@antgroup.com>
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