HDDS-9542. Ozone debug chunkinfo command shows incorrect number of entries

[aryangupta1998]

What changes were proposed in this pull request?

If we stop one or more replica datanodes of a key and then run the "ozone debug chunkinfo" command for that key then we get an Execution exception,

23/10/26 03:30:35 ERROR scm.XceiverClientGrpc: Failed to execute command GetBlock. Exception Class: java.util.concurrent.ExecutionException, Exception Message:

because we don't get any container response as we have shut down the datanode and so the pipeline is closed. The exception goes away when the dead node interval is met, i.e. after the datanode is marked closed a new pipeline is created. With this PR, we would be showing the Execution exception but will continue printing other node's getBlock result.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-9542

How was this patch tested?

Tested on a cluster.
Configs set:

"hdds.scm.replication.thread.interval": "5m",
"ozone.scm.stale.node.interval": "2m",
"ozone.scm.dead.node.interval": "4m"

When all DN's were up,

23/11/29 19:58:51 INFO impl.MetricsSystemImpl: XceiverClientMetrics metrics system started
{
  "KeyLocations": [
    [
      {
        "Datanode-HostName": "ozone-rr-5.ozone-rr.root.hwx.site",
        "Datanode-IP": "172.27.34.203",

When one of the replica DN went down,

23/11/29 20:00:05 INFO impl.MetricsSystemImpl: XceiverClientMetrics metrics system started
23/11/29 20:00:07 ERROR scm.XceiverClientGrpc: Failed to execute command GetBlock on datanode ozone-rr-4.ozone-rr.root.hwx.site Exception Class: java.util.concurrent.ExecutionException, Exception Message: org.apache.ratis.thirdparty.io.grpc.StatusRuntimeException: UNAVAILABLE: io exception
{
  "KeyLocations": [
    [
      {
        "Datanode-HostName": "ozone-rr-5.ozone-rr.root.hwx.site",
        "Datanode-IP": "172.27.34.203",
        "Container-ID": 3,

We got the exception but still printing other blocks location on remaning DN's

[sadanand48]

Thanks @aryangupta1998 , change looks good.

[sadanand48]

LGTM, pending CI

[nandakumar131]

+1, LGTM.

[sadanand48]

Thanks @aryangupta1998 for the change, @nandakumar131 for the review.