Anu Engineer created HDFS-12238:
-----------------------------------

             Summary: Ozone: Add valid trace ID check in sendCommandAsync
                 Key: HDFS-12238
                 URL: https://issues.apache.org/jira/browse/HDFS-12238
             Project: Hadoop HDFS
          Issue Type: Sub-task
          Components: ozone
    Affects Versions: HDFS-7240
            Reporter: Anu Engineer


In the function {{XceiverClientHandler#sendCommandAsync}} we should add a check 
{code}
   if(StringUtils.isEmpty(request.getTraceID())) {
      throw new IllegalArgumentException("Invalid trace ID");
    }
{code}

To ensure that ozone clients always send a valid trace ID. However, when you do 
that a set of current tests that do add a valid trace ID will fail. So we need 
to fix these tests too.

{code}
  TestContainerMetrics.testContainerMetrics
  TestOzoneContainer.testBothGetandPutSmallFile
  TestOzoneContainer.testCloseContainer
  TestOzoneContainer.testOzoneContainerViaDataNode
  TestOzoneContainer.testXcieverClientAsync
  TestOzoneContainer.testCreateOzoneContainer
  TestOzoneContainer.testDeleteContainer
  TestContainerServer.testClientServer
  TestContainerServer.testClientServerWithContainerDispatcher
  TestKeys.testPutAndGetKeyWithDnRestart
{code}

This is based on a comment from [~vagarychen] in HDFS-11580.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-dev-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-dev-h...@hadoop.apache.org

Reply via email to