Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103739428
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -53,20 +58,37 @@ shell) JAVA_OPTS=("${JAVA_OPTS[@]}"
${shellHigh_shellLow}) ;;
*) JAVA_OPTS=("${JAVA_OPTS[@]}" ${otherHigh_otherLow}) ;;
esac
-## JVM options set for logging. Review logj4 properties files to see how
they are used.
-JAVA_OPTS=("${JAVA_OPTS[@]}"
+## JVM options set for logging. Review logj4 properties files to see how
they are used.
+JAVA_OPTS=("${JAVA_OPTS[@]}"
"-Daccumulo.log.dir=${ACCUMULO_LOG_DIR}"
-
"-Daccumulo.service.id=${ACCUMULO_CMD}${ACCUMULO_SERVICE_INSTANCE}_$(hostname)"
- "-Daccumulo.audit.log=$(hostname).audit")
+
"-Daccumulo.application=${ACCUMULO_CMD}${ACCUMULO_SERVICE_INSTANCE}_$(hostname)")
case "$ACCUMULO_CMD" in
-monitor) JAVA_OPTS=("${JAVA_OPTS[@]}"
"-Dlog4j.configuration=file:${ACCUMULO_CONF_DIR}/log4j-monitor.properties") ;;
-gc|master|tserver|tracer) JAVA_OPTS=("${JAVA_OPTS[@]}"
"-Dlog4j.configuration=file:${ACCUMULO_CONF_DIR}/log4j-service.properties") ;;
-*) JAVA_OPTS=("${JAVA_OPTS[@]}"
"-Dlog4j.configuration=file:${ACCUMULO_CONF_DIR}/log4j.properties") ;;
+ monitor)
+ JAVA_OPTS=("${JAVA_OPTS[@]}"
"-Dlog4j.configuration=log4j-monitor.properties")
+ ;;
+ gc|master|tserver|tracer)
+ JAVA_OPTS=("${JAVA_OPTS[@]}"
"-Dlog4j.configuration=log4j-service.properties")
+ ;;
+ *)
+ # let log4j use its default behavior (log4j.xml, log4j.properties)
+ true
+ ;;
esac
export JAVA_OPTS
+## External class path items for Java system class loader (dependencies
not included with Accumulo)
--- End diff --
Building on the below Hadoop/ZK classpath building comment and the above
"vendor-specific releases", I *think* this will be broken with what we have in
HDP presently.
Happy to work through it and be a "guinea pig" for how we want to support
other layouts (avoiding naming the layout you're expecting now -- not sure if
it's the "Apache Hadoop fs layout" or more correctly "Apache Bigtop layout",
whatever)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
