[GitHub] accumulo pull request #245: ACCUMULO-4620 Consolidate all Maven modules into...

2017-04-07 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/245#discussion_r110495402 --- Diff: pom.xml --- @@ -80,24 +80,24 @@ ${maven.min-version} -assemble -core -docs -fate -

[GitHub] accumulo pull request #246: ACCUMULO-4620 Consolidate all Maven modules into...

2017-04-07 Thread mikewalch
GitHub user mikewalch opened a pull request: https://github.com/apache/accumulo/pull/246 ACCUMULO-4620 Consolidate all Maven modules into one directory You can merge this pull request into a Git repository by running: $ git pull https://github.com/mikewalch/accumulo

[GitHub] accumulo pull request #245: ACCUMULO-4620 Consolidate all Maven modules into...

2017-04-07 Thread mikewalch
GitHub user mikewalch opened a pull request: https://github.com/apache/accumulo/pull/245 ACCUMULO-4620 Consolidate all Maven modules into one directory This PR is for master. I am going to create PRs for 1.8 & 1.7 branches to make merging easier in the future. You can merge this

[GitHub] accumulo issue #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on the issue: https://github.com/apache/accumulo/pull/242 @milleruntime Thanks for the feedback! I'll check the JS stuff, I liked the ideas you mentioned. --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110476772 --- Diff: server/monitor/src/main/resources/resources/overview.js --- @@ -0,0 +1,293 @@ +/* +* Licensed to the Apache Software Foundation (ASF) under

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110476674 --- Diff: server/monitor/src/main/resources/resources/master.js --- @@ -0,0 +1,261 @@ +/* +* Licensed to the Apache Software Foundation (ASF) under

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110476591 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tservers/TabletServerResource.java --- @@ -0,0 +1,336 @@ +/* + * Licensed

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110468835 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/trace/TracesResource.java --- @@ -0,0 +1,372 @@ +/* + * Licensed to

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110466878 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tables/TablesResource.java --- @@ -0,0 +1,269 @@ +/* + * Licensed to

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110473250 --- Diff: server/monitor/src/main/resources/resources/master.js --- @@ -0,0 +1,261 @@ +/* +* Licensed to the Apache Software Foundation (ASF)

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110471346 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tservers/TabletServerInformation.java --- @@ -0,0 +1,143 @@ +/* + *

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110470413 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tservers/ServerShuttingDownInformation.java --- @@ -14,25 +14,32 @@ *

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110467050 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/trace/RecentTracesInformation.java --- @@ -0,0 +1,73 @@ +/* + *

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110473929 --- Diff: server/monitor/src/main/resources/resources/problems.js --- @@ -0,0 +1,232 @@ +/* +* Licensed to the Apache Software Foundation

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110473784 --- Diff: server/monitor/src/main/resources/resources/overview.js --- @@ -0,0 +1,293 @@ +/* +* Licensed to the Apache Software Foundation

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110469952 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/trace/TracesResource.java --- @@ -0,0 +1,372 @@ +/* + * Licensed to

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110471868 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/logs/DeadLoggerInformation.java --- @@ -0,0 +1,57 @@ +/* + * Licensed to

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110470726 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/XMLResource.java --- @@ -0,0 +1,67 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110468167 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/XMLInformation.java --- @@ -0,0 +1,115 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110466768 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tables/TablesResource.java --- @@ -0,0 +1,269 @@ +/* + * Licensed to

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110465485 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tables/TableInformation.java --- @@ -0,0 +1,131 @@ +/* + * Licensed to the

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110465276 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/status/StatusResource.java --- @@ -0,0 +1,116 @@ +/* + * Licensed to the

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110464940 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/tables/TableInformation.java --- @@ -0,0 +1,131 @@ +/* + * Licensed

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110464998 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/logs/LogEvent.java --- @@ -0,0 +1,57 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110464603 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/status/StatusResource.java --- @@ -0,0 +1,116 @@ +/* + * Licensed to

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110456625 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/logs/LogEvent.java --- @@ -0,0 +1,57 @@ +/* + * Licensed to the

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110454857 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/logs/LogEvent.java --- @@ -0,0 +1,57 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread milleruntime
Github user milleruntime commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110453818 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/logs/LogEvent.java --- @@ -0,0 +1,57 @@ +/* + * Licensed to the

[GitHub] accumulo pull request #244: Discuss ACCUMULO-3079: collapsing the iterator s...

2017-04-07 Thread milleruntime
GitHub user milleruntime opened a pull request: https://github.com/apache/accumulo/pull/244 Discuss ACCUMULO-3079: collapsing the iterator stack to improve performance Applied the changes from the patch submitted by Adam Fuchs on

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread lstav
Github user lstav commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110439061 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/rest/Totals.java --- @@ -0,0 +1,53 @@ +/* + * Licensed to the Apache Software

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110189876 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/view/Indexes.java --- @@ -0,0 +1,383 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110258980 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/view/Indexes.java --- @@ -0,0 +1,383 @@ +/* + * Licensed to the Apache

[GitHub] accumulo pull request #242: ACCUMULO-2181/3005 REST API and new Monitor UI

2017-04-07 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/242#discussion_r110252701 --- Diff: assemble/pom.xml --- @@ -195,6 +235,86 @@ true + org.freemarker + freemarker + +

[GitHub] accumulo pull request #240: ACCUMULO-4618 Updates from additional code revie...

2017-04-07 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/accumulo/pull/240 --- 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

[GitHub] accumulo pull request #240: ACCUMULO-4618 Updates from additional code revie...

2017-04-07 Thread mikewalch
Github user mikewalch commented on a diff in the pull request: https://github.com/apache/accumulo/pull/240#discussion_r110412136 --- Diff: maven-plugin/pom.xml --- @@ -68,6 +68,7 @@ xml-apis xml-apis + provided --- End diff --