----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69330/ -----------------------------------------------------------
Review request for oozie, András Piros and Peter Bacsko. Repository: oozie-git Description ------- Auth token cache file name should include Oozie URL, link to oozie-3379. Diffs ----- client/src/main/java/org/apache/oozie/client/AuthOozieClient.java 3a8b5ab6 core/src/main/java/org/apache/oozie/test/EmbeddedServletContainer.java fc9d840b core/src/test/java/org/apache/oozie/servlet/TestAuthFilterAuthOozieClient.java 04fde730 Diff: https://reviews.apache.org/r/69330/diff/1/ Testing ------- I added a test for authOozieClients with the different oozieUrl. I changed the code for EmbeddedServletContainer. Because this patch needs to bind the cache file to oozieUrl, you need to test whether multiple different clients can use the cache. In the original EmbeddedServletContainer code, the port was random and could not get a given container context path. So the port binding needs to be added. Thanks, zhang junfan
