[GitHub] [knox] moresandeep commented on a change in pull request #528: KNOX-2700 - Fix broken unit tests

[GitBox <gi...@apache.org>]

moresandeep commented on a change in pull request #528:
URL: https://github.com/apache/knox/pull/528#discussion_r782358264



##########
File path: gateway-test-release/webhdfs-kerb-test/src/test/resources/org/apache/knox/gateway/SecureKnoxShellTest/SecureWebHdfsPutGet.groovy
##########
@@ -30,6 +30,7 @@ session = KnoxSession.kerberosLogin(gateway, jaasConf, krb5conf, true)
 status = Hdfs.status(session).file( "/" ).now().string
 
 Hdfs.put( session ).file( file ).to( dataDir + "/" + dataFile ).now()
+Thread.sleep(1000)

Review comment:
       Thanks for the review @risdenk! We have recently started seeing this issue where `MiniDFSCluster` takes some time for the PUT operation. When I run these tests in debug mode with debugger on they work and this is where I noticed where they fail. 
   
   Are you suggesting using status to see if the file exists if not wait for given time then perform fetch operation?
   




-- 
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: dev-unsubscribe@knox.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org