ctubbsii edited a comment on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436349942
If we had a well-defined syntax for a "connectionString", it would be much
simpler: `.to(connectionString)`.
Maybe:
ctubbsii commented on issue #747: Shorten method names in AccumuloClient builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436354884
I'm willing to overlook my concerns about getting that ZK connection API
"just right" if we reserve a clean API for passing in a
keith-turner commented on a change in pull request #743: New MapReduce API
URL: https://github.com/apache/accumulo/pull/743#discussion_r231220391
##
File path:
hadoop-mapreduce/src/main/java/org/apache/accumulo/hadoop/mapreduce/InputInfo.java
##
@@ -0,0 +1,352 @@
+/*
+ *
ctubbsii commented on issue #747: Shorten method names in AccumuloClient builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436348954
> I don't think a user would ever write code like the following because they
would usually not know the number of zookeepers at compile
ctubbsii edited a comment on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436349942
If we had a well-defined syntax for a "connectionString", it would be much
simpler: `.to(connectionString)`.
Maybe:
ctubbsii edited a comment on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436349942
If we had a well-defined syntax for a "connectionString", it would be much
simpler: `.to(connectionString)`. Maybe:
keith-turner closed pull request #30: Got most ITs working
URL: https://github.com/apache/accumulo-examples/pull/30
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull
ctubbsii opened a new issue #748: Establish a "connection string" concept for
constructing Accumulo clients
URL: https://github.com/apache/accumulo/issues/748
See https://github.com/apache/accumulo/issues/747#issuecomment-436349942
The idea here is to establish a connection string
mikewalch commented on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436360444
I really like `.to(connectionString)`. I am going to create a pull request
for this issue. I think there is enough consensus. The
ctubbsii edited a comment on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436349942
If we had a well-defined syntax for a "connectionString", it would be much
simpler: `.to(connectionString)`.
Maybe:
milleruntime opened a new issue #749: Add Multi table functionality to new
MapReduce API
URL: https://github.com/apache/accumulo/issues/749
Follow on work for #743 to maintain functionality that
AccumuloMultiTableInputFormat provided in the old mapreduce API.
milleruntime commented on a change in pull request #743: New MapReduce API
URL: https://github.com/apache/accumulo/pull/743#discussion_r231240778
##
File path:
hadoop-mapreduce/src/main/java/org/apache/accumulo/hadoop/mapred/AccumuloMultiTableInputFormat.java
##
@@ -1,99
The Apache Jenkins build system has built Accumulo-Pull-Requests (build #1740)
Status: Aborted
Check console output at
https://builds.apache.org/job/Accumulo-Pull-Requests/1740/ to view the results.
ctubbsii edited a comment on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436349942
If we had a well-defined syntax for a "connectionString", it would be much
simpler: `.to(connectionString)`. Maybe
ctubbsii commented on issue #747: Shorten method names in AccumuloClient builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436349942
If we had a well-defined syntax for a "connectionString", it would be much
simpler: `.to(connectionString)`. Maybe
milleruntime commented on issue #743: New MapReduce API
URL: https://github.com/apache/accumulo/pull/743#issuecomment-436459659
I created follow on tickets so this PR doesn't become too big... @mikewalch
@ctubbsii @keith-turner OK with merging this PR as the next step in the API
The Apache Jenkins build system has built Accumulo-Pull-Requests (build #1741)
Status: Failure
Check console output at
https://builds.apache.org/job/Accumulo-Pull-Requests/1741/ to view the results.
mikewalch commented on a change in pull request #743: New MapReduce API
URL: https://github.com/apache/accumulo/pull/743#discussion_r231286641
##
File path:
hadoop-mapreduce/src/main/java/org/apache/accumulo/hadoop/mapreduce/OutputInfo.java
##
@@ -0,0 +1,147 @@
+/*
+ *
The Apache Jenkins build system has built Accumulo-Pull-Requests (build #1742)
Status: Still Failing
Check console output at
https://builds.apache.org/job/Accumulo-Pull-Requests/1742/ to view the results.
milleruntime commented on a change in pull request #743: New MapReduce API
URL: https://github.com/apache/accumulo/pull/743#discussion_r231337281
##
File path:
hadoop-mapreduce/src/main/java/org/apache/accumulo/hadoop/mapreduce/OutputInfo.java
##
@@ -0,0 +1,147 @@
+/*
+ *
mikewalch commented on issue #746: Make use of JAVA_TOOL_OPTIONS in
accumulo-env.sh
URL: https://github.com/apache/accumulo/issues/746#issuecomment-436386466
Setting `JAVA_TOOL_OPTIONS` causes a debug message that looks like it cannot
be suppressed.
```bash
$ accumulo version
mikewalch opened a new pull request #750: Updated accumulo-env.sh to include
existing CLASSPATH
URL: https://github.com/apache/accumulo/pull/750
This is an automated message from the Apache Git Service.
To respond to the
milleruntime opened a new issue #751: Deduplicate batch writer config in
MapReduce API
URL: https://github.com/apache/accumulo/issues/751
Follow on work to #743. Use the new ClientInfo object to set batch writer
properties and remove the method batchWriterOptions in OutputInfo
milleruntime commented on a change in pull request #743: New MapReduce API
URL: https://github.com/apache/accumulo/pull/743#discussion_r231321231
##
File path:
hadoop-mapreduce/src/main/java/org/apache/accumulo/hadoop/mapreduce/OutputInfo.java
##
@@ -0,0 +1,147 @@
+/*
+ *
milleruntime commented on a change in pull request #743: New MapReduce API
URL: https://github.com/apache/accumulo/pull/743#discussion_r231281567
##
File path:
hadoop-mapreduce/src/main/java/org/apache/accumulo/hadoop/mapreduce/OutputInfo.java
##
@@ -0,0 +1,147 @@
+/*
+ *
jmark99 commented on issue #708: Simplify API definition by moving impl
packages from under API packages
URL: https://github.com/apache/accumulo/issues/708#issuecomment-436263196
I will go with the xImpl. Like @ctubbsii indicated, security.crypto and
security.crypto.streams are the most
keith-turner commented on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436291393
I don't think a user would ever write code like the following because they
would usually not know the number of zookeepers at
keith-turner edited a comment on issue #747: Shorten method names in
AccumuloClient builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436290422
One reason to not use `from(String)` now is that we may want to leave it
available for future use. I have always though it
keith-turner edited a comment on issue #747: Shorten method names in
AccumuloClient builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436291844
The first time people see this API, they likely will not be writing it but
will be reading an example. I think using these
keith-turner commented on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436290422
One reason to not use `from(String)` now is that we may want to leave it
available for future use. I have always though it would
keith-turner commented on issue #747: Shorten method names in AccumuloClient
builder
URL: https://github.com/apache/accumulo/issues/747#issuecomment-436291844
The first time people see this API, they likely will not be writing it but
will be reading an example. I think using these new
mikewalch commented on a change in pull request #743: New MapReduce API
URL: https://github.com/apache/accumulo/pull/743#discussion_r231188415
##
File path:
hadoop-mapreduce/src/main/java/org/apache/accumulo/hadoop/mapreduce/OutputInfo.java
##
@@ -0,0 +1,147 @@
+/*
+ *
32 matches
Mail list logo