keith-turner closed pull request #607: ACCUMULO-4813 Add ability to provide a
load plan for bulk import
URL: https://github.com/apache/accumulo/pull/607
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
keith-turner commented on issue #607: ACCUMULO-4813 Add ability to provide a
load plan for bulk import
URL: https://github.com/apache/accumulo/pull/607#issuecomment-421389904
@ctubbsii I squashed, merged, resolved conflicts, ran mvn compile, and
pushed. I should have run mvn verify.
keith-turner commented on issue #647: question: how can I get the size of the
table in accumulo?
URL: https://github.com/apache/accumulo/issues/647#issuecomment-421402578
The accumulo shell has a `du` command that takes one or more table names as
an argument. If given multiple tables,
keith-turner commented on issue #649: TServerUtilsTest fails on master and OSX
URL: https://github.com/apache/accumulo/pull/649#issuecomment-421474077
> I assume from the approvals that y'all aren't too worried with the "why"?
Sufficiently straightforward at a glance?
My assumption
keith-turner commented on a change in pull request #648: Rename new Bulk import
API
URL: https://github.com/apache/accumulo/pull/648#discussion_r217803041
##
File path:
core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java
##
@@ -709,7 +711,7 @@
joshelser commented on issue #646: Do not require a password on the truststore
JKS
URL: https://github.com/apache/accumulo/pull/646#issuecomment-421455625
> ServerSocket[addr=0.0.0.0/0.0.0.0,localport=1024]
Seems like the newer Thrift is to blame, or just TServerUtilsTest. We only
keith-turner commented on issue #649: TServerUtilsTest fails on master and OSX
URL: https://github.com/apache/accumulo/pull/649#issuecomment-421476925
> I made the change in the SASL server binding because it did the same thing
as the BlockingServerSocket. However, the
joshelser commented on issue #649: TServerUtilsTest fails on master and OSX
URL: https://github.com/apache/accumulo/pull/649#issuecomment-421469854
FWIW, I don't see this on my linux server, and I can only assume that the
rest of you don't see this either. Your guess is as good as mine why
The Apache Jenkins build system has built Accumulo-Master (build #2464)
Status: Failure
Check console output at https://builds.apache.org/job/Accumulo-Master/2464/ to
view the results.
joshelser commented on issue #649: TServerUtilsTest fails on master and OSX
URL: https://github.com/apache/accumulo/pull/649#issuecomment-421471893
I assume from the approvals that y'all aren't too worried with the "why"?
Sufficiently straightforward at a glance?
I made the change
keith-turner commented on a change in pull request #648: Rename new Bulk import
API
URL: https://github.com/apache/accumulo/pull/648#discussion_r217801081
##
File path:
core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java
##
@@ -709,7 +711,7 @@
keith-turner commented on issue #108: Add documentation for crypto
URL: https://github.com/apache/accumulo-website/pull/108#issuecomment-421472334
Might be useful to mention SSL and Kerberose for encrypting data between
tserver and client.
milleruntime opened a new pull request #648: Rename new Bulk import API
URL: https://github.com/apache/accumulo/pull/648
Came up with this after reviewing #607
This is an automated message from the Apache Git Service.
To
joshelser commented on issue #646: Do not require a password on the truststore
JKS
URL: https://github.com/apache/accumulo/pull/646#issuecomment-421454290
PS, I'm still fighting a deterministic unit test failure in TServerUtilsTest
where the original port grabbed doesn't actually preclude
joshelser commented on issue #646: Do not require a password on the truststore
JKS
URL: https://github.com/apache/accumulo/pull/646#issuecomment-421456170
Well, at this point, this is obviously not something related to this change.
Will spin this out.
joshelser opened a new pull request #649: TServerUtilsTest fails on master and
OSX
URL: https://github.com/apache/accumulo/pull/649
A couple of tests fail in this class because the test binds to
localhost:
but then TServerSocket (via TServerUtils) binds to `0.0.0.0` which doesn't
joshelser closed pull request #646: Do not require a password on the truststore
JKS
URL: https://github.com/apache/accumulo/pull/646
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
joshelser commented on issue #646: Do not require a password on the truststore
JKS
URL: https://github.com/apache/accumulo/pull/646#issuecomment-421481292
Nevermind, you did this.
This is an automated message from the Apache
ctubbsii commented on issue #649: TServerUtilsTest fails on master and OSX
URL: https://github.com/apache/accumulo/pull/649#issuecomment-421481599
I recall we had some weird logic in there somewhere that looked for cases
where the address was "0.0.0.0" and then tried to figure out the host
joshelser commented on issue #649: TServerUtilsTest fails on master and OSX
URL: https://github.com/apache/accumulo/pull/649#issuecomment-421481765
Ok, thanks all. Will wait for Jenkins to pass then push.
This is an automated
milleruntime closed pull request #648: Rename new Bulk import API
URL: https://github.com/apache/accumulo/pull/648
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 request
joshelser closed pull request #649: TServerUtilsTest fails on master and OSX
URL: https://github.com/apache/accumulo/pull/649
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
[
https://issues.apache.org/jira/browse/ACCUMULO-4813?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Christopher Tubbs resolved ACCUMULO-4813.
-
Resolution: Fixed
> Accepting mapping file for bulk import
>
ctubbsii closed issue #647: question: how can I get the size of the table in
accumulo?
URL: https://github.com/apache/accumulo/issues/647
This is an automated message from the Apache Git Service.
To respond to the message,
ctubbsii commented on issue #646: Do not require a password on the truststore
JKS
URL: https://github.com/apache/accumulo/pull/646#issuecomment-421484340
Yeah, not a big deal to fix. I just wanted to mention it to serve a reminder
(especially to any newcomer observers). :smiley_cat:
joshelser commented on issue #649: TServerUtilsTest fails on master and OSX
URL: https://github.com/apache/accumulo/pull/649#issuecomment-421479470
> My assumption looking at the changes is that it was not binding to the
correct interface because the host was ignored. Is this correct?
joshelser commented on issue #646: Do not require a password on the truststore
JKS
URL: https://github.com/apache/accumulo/pull/646#issuecomment-421480883
Crap, I forgot about that. Will push a merge.
This is an automated
The Apache Jenkins build system has built Accumulo-Master (build #2465)
Status: Fixed
Check console output at https://builds.apache.org/job/Accumulo-Master/2465/ to
view the results.
ctubbsii commented on issue #646: Do not require a password on the truststore
JKS
URL: https://github.com/apache/accumulo/pull/646#issuecomment-421477661
I think this was cherry-picked from 1.9 to 2.0. Typically, we apply to the
oldest branch and merge forward to newer branches, to ensure
The Apache Jenkins build system has built Accumulo-Pull-Requests (build #1550)
Status: Fixed
Check console output at
https://builds.apache.org/job/Accumulo-Pull-Requests/1550/ to view the results.
30 matches
Mail list logo
