dlmarion commented on code in PR #91: URL: https://github.com/apache/accumulo-access/pull/91#discussion_r2631108622
########## README.md: ########## @@ -62,8 +62,10 @@ To run [AccessExample](src/test/java/example/AccessExample.java) ``` mvn clean package -CLASSPATH=$(ls target/accumulo-access-*.jar) java src/test/java/example/AccessExample.java -CLASSPATH=$(ls target/accumulo-access-*.jar) java src/test/java/example/AccessExample.java RED BLUE + +JAVA_TOOL_OPTIONS="--module-path=target/accumulo-access-1.0.0-SNAPSHOT.jar --add-modules=accumulo.access" java src/test/java/example/AccessExample.java Review Comment: I think this is going to require modifying accumulo-access to be a multi-module Maven project. I have no issue with this. We could likely move the antlr example to it's own module as well. @keith-turner @ctubbsii - do you have any issues with moving this to multi-module? -- 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]
