Wellington Ramos Chevreuil
Wellington Ramos Chevreuil
> Thanks for the review @wchevreuil > > Sorry, I didn't find the `ConnectionImplementation` class in the master branch. If you mean the synchronous connection, then I think the synchronous...
> Instead of Command name we need to give the classname of Test impl and it should work as is. So we need to explicitly say that in the printUsage....
> Got it, how about we update > > `Usage: hbase pe [-D]* ` to `Usage: hbase pe [-D]* ` > > And add a section for Class just below...
> Hubspot has been using this patch in production for some months. This successfully addresses the performance regression identified in [HBASE-29012](https://issues.apache.org/jira/browse/HBASE-29012), and we haven't noticed any negative side effects. Thanks...
> The PR that has added this patch seems to expect that transient is expected by default, as it added transient to several members. > > TBH I'm not sure...
> +1 with stoty, is this more like a test problem or an actual problem in main code? I have seen this manifesting on tests using specific JDK 17 implementations...
> > The JSON serialization is used in a single test in debug log output. > > No it's not, we just got another ticket that demonstrates that it's also...
> > No it's not, we just got another ticket that demonstrates that it's also called by the UI from JSP. > > We're calling GsonUtil from rits.jsp , but...
> > We've discussed this with @wchevreuil , the only sane ways seems to be removing any code that relies on serializing arbitrary multigigabyte internal objects (BucketCache) into JSON with...
> I think @stoty proposed a more general way for solving the problem that we should not print all the things in BucketCache on our web UI... > > For...