ayushtkn commented on PR #6933:
URL: https://github.com/apache/hadoop/pull/6933#issuecomment-2241535741

   Hi @szetszwo / @arp7 
   This PR seems to create some issues
   ```
   javax.security.sasl.SaslException: DIGEST-MD5: Digest-challenge format 
violation: algorithm directive missing
        at 
com.sun.security.sasl.digest.DigestMD5Client.processChallenge(DigestMD5Client.java:291)
        at 
com.sun.security.sasl.digest.DigestMD5Client.evaluateChallenge(DigestMD5Client.java:220)
        at 
org.apache.hadoop.hdfs.protocol.datatransfer.sasl.SaslParticipant.evaluateChallengeOrResponse(SaslParticipant.java:135)
        at 
org.apache.hadoop.hdfs.protocol.datatransfer.sasl.SaslDataTransferClient.doSaslHandshake(SaslDataTransferClient.java:525)
        at 
org.apache.hadoop.hdfs.protocol.datatransfer.sasl.SaslDataTransferClient.getSaslStreams(SaslDataTransferClient.java:449)
        at 
org.apache.hadoop.hdfs.protocol.datatransfer.sasl.SaslDataTransferClient.send(SaslDataTransferClient.java:292)
        at 
org.apache.hadoop.hdfs.protocol.datatransfer.sasl.SaslDataTransferClient.checkTrustAndSend(SaslDataTransferClient.java:239)
        at 
org.apache.hadoop.hdfs.protocol.datatransfer.sasl.SaslDataTransferClient.socketSend(SaslDataTransferClient.java:206)
        at 
org.apache.hadoop.hdfs.protocol.datatransfer.sasl.SaslDataTransferClient.socketSend(SaslDataTransferClient.java:196)
        at 
org.apache.hadoop.hdfs.DataStreamer.createBlockOutputStream(DataStreamer.java:1900)
        at 
org.apache.hadoop.hdfs.DataStreamer.setupPipelineForCreate(DataStreamer.java:1842)
        at org.apache.hadoop.hdfs.DataStreamer.run(DataStreamer.java:752)
   ```
   
   Can try running ``TestSaslDataTransfer`` or see the failures here:
   https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-6951/2/testReport/
   
   Moreover, this part is also bit confusing:
   ```
         if (LOG.isDebugEnabled()) {
           LOG.info("first: {}", first == null ? null : first.length == 0 ? 
"<empty>"
               : StringUtils.byteToHexString(first));
         }
   ```
   
https://github.com/apache/hadoop/pull/6933/files#diff-00d610fa84540aaa134ec7a70bd37f8a1c5726565f5a65054de5df6951484df6R526-R529
   
   Why are we checking ``isDebugEnabled`` when we are logging at ``info`` level
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to