cameronlee314 commented on a change in pull request #1173: SAMZA-2333: [AM isolation] Use cytodynamics classloader to launch job coordinator URL: https://github.com/apache/samza/pull/1173#discussion_r333255031
########## File path: gradle/dependency-versions.gradle ########## @@ -25,6 +25,7 @@ commonsCollectionVersion = "3.2.1" commonsHttpClientVersion = "3.1" commonsLang3Version = "3.4" + cytodynamicsVersion = "0.1.1-SNAPSHOT" Review comment: We don't directly bundle anything from cytodynamics (BSD2 license) inside samza. We just depend on it. Do we need to add anything? I looked at the LICENSE file, and it seems to just include certain bundled components (I think for the website?). However, it doesn't seem to have anything about libraries (e.g. kafka, apache commons, guava, etc.). ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
