cameronlee314 opened a new pull request #1498:
URL: https://github.com/apache/samza/pull/1498


   Issue: Currently, the pathing.jar file used to specify the classpath for 
run-class.sh is written directly into the top-level working directory of the 
app. When using container images, there may be some predefined permissions set 
on the working directory. This may cause permission issues with being able to 
write the pathing.jar file.
   
   Changes: Create a separate directory for writing classpath-related files 
(`manifest.txt`, `pathing.jar`) and write the `pathing.jar` to that separate 
directory. Update the classpath arguments to use the new location of the 
`pathing.jar`.
   
   Tests:
   1. `./bin/integration-tests.sh /tmp/samza-tests yarn-integration-tests 
--nopassword`
   2. Ran a job using `ProcessJobFactory` and checked that the job started up.
   
   API changes, usage/upgrade instructions: 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.

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


Reply via email to