abstractdog commented on PR #493:
URL: https://github.com/apache/tez/pull/493#issuecomment-4415809969

   > @abstractdog , can you please remove the JAVA_HOME env varaible setting in 
docke run command from Jenkins? I tried creating symlink but its not helping.
   > 
   > Let me know if i need to revert #494.
   > 
   > CI for tez is not a good experiece at all, this issue should have been 
caught in #494 itself. Moreover, not sure why mvninstall is failing in run 4. 
This diff patch running is not helping as it is trying to dowload the SNAPSHOT 
tez modules jars (i believe).
   
   this is very weird, I guess your Dockerfile change didn't trigger a 
precommit test
   I'm in favor of running the whole test for every PR, I already created a 
corresponding ticket for that https://issues.apache.org/jira/browse/TEZ-4355
   
   your improvement in #494 was great, so I would not revert it if we can fix 
the problem in 1-2 days (considering the project's pace, it's fine)


-- 
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]

Reply via email to