Details
-
Bug
-
Status: Resolved
-
Minor
-
Resolution: Fixed
-
2.0.0
-
None
-
None
-
Reviewed
Description
If region disables the stats, it won’t response any ClientProtos.RegionLoadStats to client. So the NPE will happen in AsyncRequestFutureImpl#updateStats.
We should use relevant log instead of NPE because the data manipulation shouldn’t be broken by statistics.
protected void updateStats(ServerName server, Map<byte[], MultiResponse.RegionResult> results) { … ClientProtos.RegionLoadStats stat = regionStats.getValue().getStat(); RegionLoadStats regionLoadstats = ProtobufUtil.createRegionLoadStats(stat); … }