jnturton opened a new pull request #2434:
URL: https://github.com/apache/drill/pull/2434


   ## Description
   
   The removed test is a copy pasta of [the log dir test directly above 
it](https://github.com/apache/drill/blob/4aefcef2b665c5737471664912a26ef6ed9a6cfc/drill-yarn/src/test/java/org/apache/drill/yarn/scripts/TestScripts.java#L253)
 so does not test
   the native lib dir variables it claims to at all.  There is a legitimate 
test of the native lib 
   dir env vars present in the same test class.
   
   ## Documentation
   N/A
   
   ## Testing
   N/A
   


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