Duong Nguyen
Duong Nguyen
+1. At least we should have something to tell outside when the model has been updated. Need a ng-change here.
Thanks for the patch @Galsza. LGTM, just one nitpicking comment inline.
> There is a call to looupKey here: > > https://github.com/apache/ozone/blob/1ea4c2adc640a02508696bf346c7f1f7b6e0afd9/hadoop-ozone/client/src/main/java/org/apache/hadoop/ozone/client/rpc/RpcClient.java#L1264 > > Should that be changed to getKeyInfo()? Yes, I cleaned that up as well. > There is a...
> e by default. We already demonstrate that it increase the number of ops the S3G can handle and the user can still disable it if they want. Enabling it...
> Code can continue testing and add a metric at the end for successful reads vs. not found read. It is a valid test to look at the performance of...
As @errose28 already pointed out as per HDDS-6931, the common theme of the problems is: pre-process validation should only perform logic on data which is not supposed to change between...
+1 on either supporting lock fainess or making `Striped#custom` public.
> We're going to add something that can be used for this (by adding `Striped.custom`, similar to #1893 (though that requested `lazyWeakCustom`, which we aren't doing until we have a...
@szetszwo