Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/227#discussion_r104454813
--- Diff:
server/tserver/src/main/java/org/apache/accumulo/tserver/NativeMap.java ---
@@ -60,39 +60,44 @@
* would be a mistake for long lived
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo/pull/227
I am going to squash and merge today if no one objects.
---
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
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/227#discussion_r104438882
--- Diff:
server/tserver/src/main/java/org/apache/accumulo/tserver/NativeMap.java ---
@@ -60,39 +60,44 @@
* would be a mistake for long lived
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/227#discussion_r104434464
--- Diff: assemble/bin/accumulo-util ---
@@ -411,12 +411,16 @@ function create_config() {
}
function build_native
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/227#discussion_r104434490
--- Diff: assemble/bin/accumulo-util ---
@@ -411,12 +411,16 @@ function create_config() {
}
function build_native
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/227#discussion_r104210324
--- Diff:
server/tserver/src/main/java/org/apache/accumulo/tserver/NativeMap.java ---
@@ -70,11 +70,10 @@
static {
// Check standard
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/227
ACCUMULO-4596 Stoped using ACCUMULO_HOME for native libs
* Using system property rather than environment variable to configure
native libraries.
* Removed unnnecessary system property
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/226#discussion_r103973200
--- Diff: assemble/bin/accumulo ---
@@ -72,13 +72,6 @@ function main() {
export HADOOP_HOME=$HADOOP_PREFIX
export HADOOP_HOME_WARN_SUPPRESS
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/225#discussion_r103942407
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/metrics/MetricsConfiguration.java
---
@@ -148,46 +148,35 @@ public Configuration
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/226
ACCUMULO-4594 Fix Kerberos documentation
* Added documentation for configuring Kerberos using 2.0.0 scripts
* Removed kerberos config file references from accumulo command as
these can
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/225#discussion_r103797048
--- Diff: docs/src/main/asciidoc/chapters/administration.txt ---
@@ -571,17 +571,20 @@ which allows for routing of metrics generated by
registered
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/225#discussion_r103791991
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/metrics/MetricsConfiguration.java
---
@@ -148,46 +148,35 @@ public Configuration
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/225#discussion_r103792135
--- Diff: docs/src/main/asciidoc/chapters/administration.txt ---
@@ -571,17 +571,20 @@ which allows for routing of metrics generated by
registered
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/225#discussion_r103789283
--- Diff: assemble/bin/accumulo-util ---
@@ -298,7 +298,7 @@ function create_config() {
MAVEN_PROJ_BASEDIR="\
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/225
ACCUMULO-4593 Move metrics config to templates dir
* Updated documentation to reflect change to templates dir
* Legacy metrics now look for configuration file on classpath rather
than
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/224#discussion_r103748199
--- Diff: docs/src/main/asciidoc/chapters/summaries.txt ---
@@ -0,0 +1,209 @@
+// Licensed to the Apache Software Foundation (ASF) under one or more
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/224#discussion_r103750219
--- Diff: docs/src/main/asciidoc/chapters/summaries.txt ---
@@ -0,0 +1,209 @@
+// Licensed to the Apache Software Foundation (ASF) under one or more
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/224#discussion_r103737501
--- Diff: docs/src/main/asciidoc/chapters/summaries.txt ---
@@ -0,0 +1,209 @@
+// Licensed to the Apache Software Foundation (ASF) under one or more
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
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
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[@]}"
${shellHig
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
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
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
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo/pull/221
I would like to merge this today. Let me know if there is anything else
that needs to be changed.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103504035
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/Accumulo.java ---
@@ -108,69 +105,20 @@ public static synchronized Path
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103497233
--- Diff: assemble/conf/log4j-monitor.properties ---
@@ -15,18 +15,18 @@
# Write out everything at the DEBUG level to the debug log
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103496710
--- Diff: assemble/conf/log4j-monitor.properties ---
@@ -15,18 +15,18 @@
# Write out everything at the DEBUG level to the debug log
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103484745
--- Diff:
test/src/main/java/org/apache/accumulo/test/functional/MonitorLoggingIT.java ---
@@ -1,121 +0,0 @@
-/*
- * Licensed to the Apache
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103227251
--- Diff:
test/src/main/java/org/apache/accumulo/test/functional/MonitorLoggingIT.java ---
@@ -1,121 +0,0 @@
-/*
- * Licensed to the Apache
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103023673
--- Diff:
test/src/main/java/org/apache/accumulo/test/functional/MonitorLoggingIT.java ---
@@ -1,121 +0,0 @@
-/*
- * Licensed to the Apache
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103021273
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -25,6 +25,21 @@
# Variables that must be set
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo/pull/221
> One open-ended question: how can we make sure that log-forwarding works
with the removal of MonitorLoggingIT? Do we need to extend MAC to allow us to
configure log4j better and t
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103013232
--- Diff: assemble/conf/log4j-monitor.properties ---
@@ -36,6 +36,17 @@
log4j.appender.GUI=org.apache.accumulo.server.monitor.LogService
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103013138
--- Diff:
server/base/src/main/java/org/apache/accumulo/server/Accumulo.java ---
@@ -108,69 +105,20 @@ public static synchronized Path
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103011353
--- Diff: assemble/conf/log4j-service.properties ---
@@ -15,42 +15,48 @@
# Write out everything at the DEBUG level to the debug log
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103003251
--- Diff: assemble/conf/templates/accumulo-env.sh ---
@@ -25,6 +25,21 @@
# Variables that must be set
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103003156
--- Diff:
start/src/test/java/org/apache/accumulo/start/util/AsyncSocketAppenderTest.java
---
@@ -36,7 +31,7 @@
@Before
public void
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r103000929
--- Diff: assemble/conf/log4j-monitor.properties ---
@@ -36,6 +36,17 @@
log4j.appender.GUI=org.apache.accumulo.server.monitor.LogService
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/221#discussion_r102999845
--- Diff: assemble/conf/log4j-service.properties ---
@@ -15,42 +15,48 @@
# Write out everything at the DEBUG level to the debug log
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/221
ACCUMULO-4588 Simplify Accumulo logging configuration
* Removed log4j config being done in Java but some remains
* Logging is now configured using standard log4j JVM property
'log4
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo/pull/218
I talked to @milleruntime offline about accumulo-env.sh rename. For this
PR, I decided to not do the rename as concerns were brought up about changing
name to accumulo.conf. In the end, I
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo-examples/pull/4
Updates due to tool.sh being moved to 'accumulo-util hadoop-jar' in Aâ¦
â¦CCUMULO-4528
You can merge this pull request into a Git repository by running:
$ git pull https
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo/pull/218
Thanks everyone for reviewing! Your feedback resulted in a lot of changes
that made this PR better. I think I have addressed all comments. Unless
someone needs more time to review or wants
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/218#discussion_r101785204
--- Diff: INSTALL.md ---
@@ -76,19 +76,18 @@ When configuring Accumulo the following information
about these dependencies
must be provided
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/218#discussion_r101784212
--- Diff: assemble/conf/examples/generic_logger.properties ---
@@ -24,7 +24,7 @@ log4j.appender.A2.layout.ConversionPattern=%d{ISO8601}
[%-8c{2}] %-5p
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/218#discussion_r101784185
--- Diff: assemble/conf/examples/generic_logger.properties ---
@@ -15,7 +15,7 @@
# Write out everything at the DEBUG level to the debug log
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/218#discussion_r101782473
--- Diff: INSTALL.md ---
@@ -40,13 +40,13 @@ Accumulo has some optional native code that improves
its performance and
stability. Before configuring
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/218#discussion_r101610417
--- Diff: assemble/contrib/tool.sh ---
@@ -26,14 +26,21 @@ contrib=$( cd -P "$( dirname "$SOURCE" )" && pwd )
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/218#discussion_r101609853
--- Diff: INSTALL.md ---
@@ -40,13 +40,13 @@ Accumulo has some optional native code that improves
its performance and
stability. Before configuring
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/218#discussion_r101530931
--- Diff: INSTALL.md ---
@@ -25,11 +25,11 @@ source code. Unpack as follows.
tar xzf /accumulo-X.Y.Z-bin.tar.gz
cd accumulo-X.Y.Z
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/218#discussion_r101365500
--- Diff: INSTALL.md ---
@@ -57,15 +57,15 @@ The script will ask you questions about your set up.
Below are some suggestions:
processes like
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/218
ACCUMULO-4528 Accumulo scripts improvements
* Accumulo now determines hostname in Java
* Minimized use of environment variables
* Consolidated scripts by moving service.sh and cluster.sh
Github user mikewalch closed the pull request at:
https://github.com/apache/accumulo/pull/213
---
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
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo/pull/213
Closing for now. I want to add some more work to this and open up new PR.
---
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
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo-testing/pull/4
ACCUMULO-4581 Fixing several bugs in agitator scripts
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mikewalch/accumulo-testing
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/213
ACCUMULO-4528 Accumulo scripts improvements
* Accumulo now determines hostname in Java
* Minimized use of environment variables
* Consolidated scripts by moving service.sh and cluster.sh
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/3#discussion_r98955888
--- Diff: core/src/main/java/org/apache/accumulo/testing/core/TestEnv.java
---
@@ -96,15 +97,22 @@ public String getPid
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/3#discussion_r98556591
--- Diff: core/src/main/java/org/apache/accumulo/testing/core/TestEnv.java
---
@@ -96,15 +97,22 @@ public String getPid
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo-testing/pull/3
ACCUMULO-4579 Fixed hadoop config bug in accumulo-testing
* TestEnv now returns Hadoop configuration that is loaded from config files
but expects HADOOP_PREFIX to be set
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/2#discussion_r97820268
--- Diff:
core/src/main/java/org/apache/accumulo/testing/core/continuous/ContinuousIngest.java
---
@@ -183,8 +177,9 @@ private static long flush
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/2#discussion_r97820053
--- Diff: README.md ---
@@ -46,9 +46,88 @@ walkers in 5 containers in YARN using the Image.xml
module.
./bin/accumulo-testing rw
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/2#discussion_r97819994
--- Diff: README.md ---
@@ -32,9 +32,9 @@ on your machine as well as an Accumulo instance to use
for testing.
The random walk test generates
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/205
ACCUMULO-4510 Removed continuous ingest code
* Code was moved to accumulo-testing repo
* Kept copy of ContinuousIngest class as its used by other tests
This PR will be merged after
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo-testing/pull/2
ACCUMULO-4510 Refactored Continous Ingest tests
* Continuous ingest applications can be launched locally or in YARN and
are now configured by accumulo-testing.properties file
Github user mikewalch closed the pull request at:
https://github.com/apache/accumulo-examples/pull/3
---
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
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo-examples/pull/3
ACCUMULO-4566 Removed code not used by any example
* Code was accidentally moved
* Cleaned up dependencies after running 'mvn dependency:analyze'
You can merge this pull request
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo-testing/pull/1
I think that this pull request is ready to merged. Before this is merged, I
want to mention two more commits that were added to PR in case any one wants to
review:
* 45b9dbfbbcaf
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r95236835
--- Diff: conf/accumulo-testing-env.sh.example ---
@@ -0,0 +1,25 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r95201587
--- Diff: README.md ---
@@ -1 +1,54 @@
# Apache Accumulo Testing Suite
+
+The Apache Accumulo testing suite contains applications
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94992878
--- Diff: bin/accumulo-testing ---
@@ -0,0 +1,89 @@
+#! /usr/bin/env bash
+
+# Licensed to the Apache Software Foundation (ASF) under
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94991969
--- Diff: bin/accumulo-testing ---
@@ -0,0 +1,89 @@
+#! /usr/bin/env bash
+
+# Licensed to the Apache Software Foundation (ASF) under
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94629586
--- Diff: conf/accumulo-testing-env.sh.example ---
@@ -0,0 +1,25 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94629706
--- Diff: bin/accumulo-testing ---
@@ -0,0 +1,89 @@
+#! /usr/bin/env bash
+
+# Licensed to the Apache Software Foundation (ASF) under
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94629162
--- Diff:
twill/src/main/java/org/apache/accumulo/testing/twill/TestRunner.java ---
@@ -0,0 +1,166 @@
+/*
+ * Licensed to the Apache
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94983756
--- Diff: bin/accumulo-testing ---
@@ -0,0 +1,89 @@
+#! /usr/bin/env bash
+
+# Licensed to the Apache Software Foundation (ASF) under
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94629932
--- Diff: bin/accumulo-testing ---
@@ -0,0 +1,89 @@
+#! /usr/bin/env bash
+
+# Licensed to the Apache Software Foundation (ASF) under
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94629394
--- Diff:
twill/src/main/java/org/apache/accumulo/testing/twill/TestRunner.java ---
@@ -0,0 +1,166 @@
+/*
+ * Licensed to the Apache
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94621811
--- Diff: bin/accumulo-testing ---
@@ -0,0 +1,89 @@
+#! /usr/bin/env bash
+
+# Licensed to the Apache Software Foundation (ASF) under
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94621759
--- Diff: README.md ---
@@ -1 +1,50 @@
# Apache Accumulo Testing Suite
+
+The Apache Accumulo testing suite contains applications
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-testing/pull/1#discussion_r94621728
--- Diff: README.md ---
@@ -1 +1,50 @@
# Apache Accumulo Testing Suite
+
+The Apache Accumulo testing suite contains applications
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/197
ACCUMULO-4510 Removing random walk test code
* Code was moved to new accumulo-testing repo
See apache/accumulo-testing#1 for pull request to add code to
accumulo-testing repo.
You can
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo-testing/pull/1
Accumulo 4510
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mikewalch/accumulo-testing accumulo-4510
Alternatively you can review
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/194#discussion_r93795541
--- Diff: assemble/src/main/assemblies/component.xml ---
@@ -40,15 +40,48 @@
org.apache.commons:commons-math3
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/194#discussion_r93785751
--- Diff: assemble/src/main/assemblies/component.xml ---
@@ -40,15 +40,48 @@
org.apache.commons:commons-math3
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/194#discussion_r93786163
--- Diff:
server/monitor/src/main/resources/templates/org/apache/accumulo/monitor/rest/view/Overview/index.ftl
---
@@ -0,0 +1,83
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-examples/pull/2#discussion_r92211623
--- Diff: examples.conf.template ---
@@ -0,0 +1,31 @@
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-examples/pull/2#discussion_r92212168
--- Diff: README.md ---
@@ -34,15 +34,20 @@ Before running any of the examples, the following steps
must be performed.
git clone
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-examples/pull/2#discussion_r92210797
--- Diff: docs/client.md ---
@@ -28,13 +28,12 @@ Using the accumulo command, you can run the simple
client examples by providing
class name
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-examples/pull/1#discussion_r91751254
--- Diff: run.sh ---
@@ -0,0 +1,21 @@
+#! /usr/bin/env bash
+
+# Licensed to the Apache Software Foundation (ASF) under one or more
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo-examples/pull/1#discussion_r91743309
--- Diff: README.md ---
@@ -20,64 +20,65 @@ limitations under the License.
Before running any of the examples, the following steps must
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo-examples/pull/1
See https://github.com/apache/accumulo/pull/191 for corresponding PR to
remove examples from Accumulo repo
---
If your project is set up for it, you can reply to this email and have your
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo/pull/191
See https://github.com/apache/accumulo-examples/pull/1 for corresponding PR
to add examples to new repo.
---
If your project is set up for it, you can reply to this email and have your
reply
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/191
ACCUMULO-4511 Removed Accumulo Examples
* Moved code and documentation to new accumulo-examples repo
* Updated references in user manual
* Update README.md to reflect the change
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/190#discussion_r91133551
--- Diff: docs/src/main/resources/examples/index.md ---
@@ -0,0 +1,99 @@
+
+---
+title: Apache Accumulo Examples
+---
+
+## Setup
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/190#discussion_r91128687
--- Diff: docs/src/main/resources/timestamps.html ---
@@ -1,160 +0,0 @@
-
-
--- End diff --
For every HTML file, similar content
Github user mikewalch commented on a diff in the pull request:
https://github.com/apache/accumulo/pull/190#discussion_r91127750
--- Diff: docs/src/main/resources/examples/index.md ---
@@ -0,0 +1,99 @@
+
+---
+title: Apache Accumulo Examples
+---
+
+## Setup
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/190
Documentation updates
* Removed old HTML documentation. Did work in multiple commits to simplify
review.
* Improved docs of examples
You can merge this pull request into a Git repository
Github user mikewalch commented on the issue:
https://github.com/apache/accumulo/pull/189
I updated the PR and stripped out the code used to generate html.
---
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
GitHub user mikewalch opened a pull request:
https://github.com/apache/accumulo/pull/189
ACCUMULO-4530 Stop generating docs/config.html
* Configuration docs can be found in appendix of user manual
* Updated all pointers to config.html to user manual instead
* Kept code
101 - 200 of 291 matches
Mail list logo