GitHub user ctubbsii opened a pull request:
https://github.com/apache/accumulo/pull/223
ACCUMULO-4409 Create AccumuloMonitorAppender
Creates an AccumuloMonitorAppender, and does additional logging system
cleanup in the files and scripts.
You can merge this pull request into a Git r
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103702178
--- Diff: assemble/conf/log4j-monitor.properties ---
@@ -16,23 +16,21 @@
## Log4j 1.2 file that configures logging for Accumulo Monitor
## The s
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103705868
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/Accumulo.java ---
@@ -107,18 +104,6 @@ public static synchronized Path
getAccumuloInstanc
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103707238
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to t
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103706254
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/watcher/Log4jConfiguration.java
---
@@ -1,59 +0,0 @@
-/*
- * Licensed to the Apac
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103702937
--- Diff: assemble/conf/log4j-service.properties ---
@@ -13,43 +13,47 @@
# See the License for the specific language governing permissions and
#
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r10370
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -53,20 +58,37 @@ shell) JAVA_OPTS=("${JAVA_OPTS[@]}"
${shellHigh_shellLow}) ;;
*) J
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103718041
--- Diff: assemble/bin/accumulo ---
@@ -127,13 +127,12 @@ function main() {
JAVA=($ACCUMULO_JAVA_PREFIX $JAVA)
fi
-
CLASSPAT
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103719146
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to t
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103718100
--- Diff: assemble/conf/log4j-monitor.properties ---
@@ -16,23 +16,21 @@
## Log4j 1.2 file that configures logging for Accumulo Monitor
## The s
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103720637
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to t
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103718621
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -53,20 +58,37 @@ shell) JAVA_OPTS=("${JAVA_OPTS[@]}"
${shellHigh_shellLow}) ;;
*) J
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103732845
--- Diff: assemble/conf/log4j-monitor.properties ---
@@ -16,23 +16,21 @@
## Log4j 1.2 file that configures logging for Accumulo Monitor
## The sy
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103733316
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -53,20 +58,37 @@ shell) JAVA_OPTS=("${JAVA_OPTS[@]}"
${shellHigh_shellLow}) ;;
*) J
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103733722
--- Diff: assemble/conf/log4j-service.properties ---
@@ -13,43 +13,47 @@
# See the License for the specific language governing permissions and
#
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103734610
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to th
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103735127
--- Diff: assemble/bin/accumulo ---
@@ -127,13 +127,12 @@ function main() {
JAVA=($ACCUMULO_JAVA_PREFIX $JAVA)
fi
-
CLASSPATH
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103733871
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -53,20 +58,37 @@ shell) JAVA_OPTS=("${JAVA_OPTS[@]}"
${shellHigh_shellLow}) ;;
*) JA
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103736418
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -53,20 +58,37 @@ shell) JAVA_OPTS=("${JAVA_OPTS[@]}"
${shellHigh_shellLow}) ;;
*) JA
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103737450
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to th
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103738255
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to th
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103738878
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -53,20 +58,37 @@ shell) JAVA_OPTS=("${JAVA_OPTS[@]}"
${shellHigh_shellLow}) ;;
*) J
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103739128
--- Diff: assemble/conf/log4j-service.properties ---
@@ -13,43 +13,47 @@
# See the License for the specific language governing permissions and
#
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}) ;;
*) J
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103739634
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to t
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103739661
--- Diff: assemble/conf/log4j.properties ---
@@ -13,22 +13,24 @@
# See the License for the specific language governing permissions and
# limitati
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103740091
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to t
Github user joshelser commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103740195
--- Diff: assemble/conf/log4j.properties ---
@@ -13,22 +13,24 @@
# See the License for the specific language governing permissions and
# limitat
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103742009
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to th
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103742376
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to th
Github user keith-turner commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103981148
--- Diff: assemble/conf/log4j-service.properties ---
@@ -13,43 +13,47 @@
# See the License for the specific language governing permissions and
Github user keith-turner commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r103985292
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed t
Github user ctubbsii commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/223#discussion_r104052130
--- Diff:
server/monitor/src/main/java/org/apache/accumulo/monitor/util/AccumuloMonitorAppender.java
---
@@ -0,0 +1,153 @@
+/*
+ * Licensed to th
Github user asfgit closed the pull request at:
https://github.com/apache/accumulo/pull/223
---
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 en
34 matches
Mail list logo