Filed https://issues.apache.org/jira/browse/HBASE-22564
张铎(Duo Zhang) <palomino...@gmail.com> 于2019年6月11日周二 下午3:53写道: > Let me do a YCSB test about the performance. > > Stack <st...@duboce.net> 于2019年6月11日周二 下午1:15写道: > >> +1 on merge from me. >> >> It removes the complicated multi-threaded edifice we'd built client-side >> to >> fake an async behavior replacing it with an actual async implementation. >> Users will immediately notice a radical plummet in working thread count on >> the client side. >> >> For the cleanup of old idioms alone, in test code in particular, the patch >> is worth merging. >> >> Any perf numbers to share comparing old sync and async? >> >> What about difference in operation? Is there any commentary or doc or >> release note to point at? >> >> Thanks, >> S >> >> >> >> On Mon, Jun 10, 2019 at 6:59 PM 张铎(Duo Zhang) <palomino...@gmail.com> >> wrote: >> >> > https://issues.apache.org/jira/browse/HBASE-21512 >> > >> > "Reimplement sync client based on async client" >> > >> > The jira title tells everything. This is what I promised when I first >> > introduced the async client in HBase, about three years ago, that the >> sync >> > client can be implemented on top of the async client, so we can remove >> the >> > old sync client implementation, which can reduce our client code base a >> > lot. >> > >> > I've already opened a PR here, and received several feedback(thanks >> stack!) >> > >> > https://github.com/apache/hbase/pull/287 >> > >> > It shows that we add 8,663 lines and remove 31,386 lines. >> > >> > This is the flaky dashboard for this branch >> > >> > >> > >> https://builds.apache.org/job/HBASE-Find-Flaky-Tests/job/HBASE-21512/lastSuccessfulBuild/artifact/dashboard.html >> > >> > With the recent efforts I think it is getting better. >> > >> > Will fill the release note soon, it will be a fat one. >> > >> > Please vote >> > >> > [] +1 >> > [] +0/-0 >> > [] -1 Do not merge the branch back because ... >> > >> > Thanks. Any suggestions are welcomed. >> > >> >