dlmarion commented on issue #3087:
URL: https://github.com/apache/accumulo/issues/3087#issuecomment-1398788327
Closing for now, can be re-opened if needed.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above
dlmarion closed issue #3087: Client in endless loop when ZooKeeper unavailable
URL: https://github.com/apache/accumulo/issues/3087
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment
dlmarion commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1398803887
Could this be the cause of #2606 ?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the sp
EdColeman commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1398817770
It does not seem to resolve #2606, I ran it with this change and the test
still fails. @DomGarguilo noted that the slow iterator is pausing between calls
to next which may be different
dlmarion commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1398819178
ok, thx for checking.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific co
EdColeman commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1398893342
30c674cbd8 modifies the ScanServerIT to test scan timeout - may resolve #2606
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to
DomGarguilo merged PR #59:
URL: https://github.com/apache/accumulo-proxy/pull/59
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr.
dlmarion opened a new pull request, #3167:
URL: https://github.com/apache/accumulo/pull/3167
Closes #3062
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe
keith-turner opened a new pull request, #3168:
URL: https://github.com/apache/accumulo/pull/3168
…uplicate data.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscr
cshannon commented on PR #3136:
URL: https://github.com/apache/accumulo/pull/3136#issuecomment-1399303663
@ctubbsii and @dlmarion - I just pushed a new update and I have
significantly reworked the PR. There's still work to be done...comments need
updating, some tests still need fixing and t
cshannon commented on issue #608:
URL: https://github.com/apache/accumulo/issues/608#issuecomment-1399316343
I investigated this again yesterday and there are still some reports of this
same behavior happening against 1.10.x. It's similar behavior where there is
missing data returned from t
keith-turner commented on PR #3164:
URL: https://github.com/apache/accumulo/pull/3164#issuecomment-1399325525
> @keith-turner - you may want to merge this directly to 2.1
That will be a lot cleaner than cherry picking it back and then merging it
forward. I am going to do that.
--
T
keith-turner merged PR #3164:
URL: https://github.com/apache/accumulo/pull/3164
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr..
keith-turner commented on code in PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#discussion_r1083513557
##
server/compaction-coordinator/src/main/java/org/apache/accumulo/coordinator/CompactionCoordinator.java:
##
@@ -319,39 +325,38 @@ public void run() {
}
keith-turner commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1399573166
> In IsolatedScanner, the two timeouts are confusing and it could be an
error that there are two values when one was intended, but I preserved current
behavior.
@EdColeman c
Manno15 commented on issue #113:
URL:
https://github.com/apache/accumulo-examples/issues/113#issuecomment-1399592006
After coming back to this, I have been able to reproduce the error. I am
still looking for a solution but my guess is the client that was used to create
the viewer was close
dlmarion commented on code in PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#discussion_r1083988954
##
server/compaction-coordinator/src/main/java/org/apache/accumulo/coordinator/CompactionCoordinator.java:
##
@@ -319,39 +325,38 @@ public void run() {
}
dlmarion commented on code in PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#discussion_r1084006624
##
server/manager/src/main/java/org/apache/accumulo/manager/Manager.java:
##
@@ -217,6 +217,8 @@ public class Manager extends AbstractServer
private final AtomicBo
dlmarion commented on code in PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#discussion_r1084007998
##
server/manager/src/main/java/org/apache/accumulo/manager/Manager.java:
##
@@ -1012,16 +1013,19 @@ private SortedMap
gatherTableInformation(
badServer
dlmarion commented on code in PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#discussion_r1084008491
##
server/compaction-coordinator/src/main/java/org/apache/accumulo/coordinator/CompactionCoordinator.java:
##
@@ -319,39 +325,38 @@ public void run() {
}
dlmarion commented on issue #608:
URL: https://github.com/apache/accumulo/issues/608#issuecomment-1400299506
Another thought is that `LinkingIterator.goodTransition` defaults to
returning `true` and only returning `false` under certain conditions. I'm
wondering if that should be reversed, d
dlmarion commented on code in PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#discussion_r1084006976
##
server/manager/src/main/java/org/apache/accumulo/manager/Manager.java:
##
@@ -1251,6 +1258,15 @@ boolean canSuspendTablets() {
throw new IllegalStateExcepti
ddanielr opened a new pull request, #3169:
URL: https://github.com/apache/accumulo/pull/3169
WIP for tests
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e
ddanielr closed pull request #3169: Bugfix/check property values backend
URL: https://github.com/apache/accumulo/pull/3169
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To un
EdColeman commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1400652298
@keith-turner Basically there are two timeouts in IsolatedScanner - one is
inherited by extending ScannerOptions and the other is local to IsolatedScanner
(now named iteratorTimeout in
EdColeman commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1400834950
As @DomGarguilo noted in https://github.com/apache/accumulo/issues/2606 -
I'm not sure that the timeouts being tested are actually the scanner timeouts.
There seems to be a sensitivit
EdColeman commented on code in PR #3146:
URL: https://github.com/apache/accumulo/pull/3146#discussion_r1084461119
##
shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java:
##
@@ -353,6 +344,31 @@ public int execute(final String fullCommand, final
CommandLine
EdColeman commented on code in PR #3146:
URL: https://github.com/apache/accumulo/pull/3146#discussion_r1084464373
##
shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java:
##
@@ -353,6 +344,31 @@ public int execute(final String fullCommand, final
CommandLine
dlmarion commented on code in PR #3168:
URL: https://github.com/apache/accumulo/pull/3168#discussion_r1085247845
##
core/src/main/java/org/apache/accumulo/core/client/impl/TabletLocatorImpl.java:
##
@@ -330,8 +348,16 @@ else if (!lookupFailed)
tabletLocations.add(tl);
dlmarion commented on code in PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#discussion_r1085464357
##
server/manager/src/main/java/org/apache/accumulo/manager/Manager.java:
##
@@ -950,11 +952,10 @@ private SortedMap
gatherTableInformation(
Set currentServer
keith-ratcliffe opened a new issue, #3170:
URL: https://github.com/apache/accumulo/issues/3170
**Describe the bug**
In prior versions of Accumulo, the user could copy and paste a
newline-separated list of commands into the shell and have them be executed in
sequence. In 2.1, this see
dlmarion commented on PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#issuecomment-1402484884
There is something in the PR that is causing the balancer ITs to fail. I
can't figure out what it is, so I'm going to let this sit for a while and close
later if we can't figure it out.
DomGarguilo commented on PR #23:
URL: https://github.com/apache/accumulo-proxy/pull/23#issuecomment-1402557902
It seems like there are some good improvements here. If there is no activity
on this PR I may open a new one and try to incorporate some of these changes at
some point.
--
This
dlmarion commented on issue #3170:
URL: https://github.com/apache/accumulo/issues/3170#issuecomment-1402619461
I was able to reproduce your issue. IIRC the jline dependency was upgraded
and looking at their documentation, issues, and user list, I'm guessing this is
the issue. I did notice t
DomGarguilo opened a new issue, #66:
URL: https://github.com/apache/accumulo-proxy/issues/66
In #59, the way a user authenticates to the proxy was changed. This changes
some things that are used in `proxy.properties`. For example, the proxy looks
for a property with the key `sharedSecret` w
dlmarion commented on issue #3170:
URL: https://github.com/apache/accumulo/issues/3170#issuecomment-1402647258
Reading through some of the issues and comments on the user list, it looks
like we might need to create out own parser.
--
This is an automated message from the Apache Git Servic
dlmarion opened a new pull request, #3171:
URL: https://github.com/apache/accumulo/pull/3171
Closes #3170
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe
dlmarion commented on PR #3171:
URL: https://github.com/apache/accumulo/pull/3171#issuecomment-1402765491
It appears that JLine implemented (and enabled by default)
[bracketed-paste](https://en.wikipedia.org/wiki/Bracketed-paste). This PR
disables that and disables highlighting. I tested th
EdColeman commented on issue #2606:
URL: https://github.com/apache/accumulo/issues/2606#issuecomment-1402836124
The javadoc in ScannerBase says that the timeout is a retry timeout.
```
This setting determines how long a scanner will automatically retry when a
failure occurs. By
EdColeman commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1402838794
Closing this and will submit a new PR against 2.1. After more investigation
timeOut should be renamed retryTimeout and there may be other changes that can
provide readability improvem
EdColeman closed pull request #3166: rename ScannerOptions timeOut to
scanTimeOut, fix possible bug
URL: https://github.com/apache/accumulo/pull/3166
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to
EdColeman opened a new pull request, #3172:
URL: https://github.com/apache/accumulo/pull/3172
Fixes parameter validation of timeout where the instance variable timeOut
was used.
Renames instance variable timeOut to retryTimeout to reflect the intended
purpose in the javadoc.
This
ddanielr commented on code in PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#discussion_r1086106035
##
core/src/main/java/org/apache/accumulo/core/client/ClientSideIteratorScanner.java:
##
@@ -213,7 +213,7 @@ public ClientSideIteratorScanner(final Scanner scanner) {
ddanielr commented on PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#issuecomment-1402915758
:+1: Changing the verbiage to `retryTimeout` makes its use much clearer!
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitH
dlmarion commented on PR #3171:
URL: https://github.com/apache/accumulo/pull/3171#issuecomment-1403474518
Full IT build passed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment
dlmarion merged PR #3171:
URL: https://github.com/apache/accumulo/pull/3171
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@ac
dlmarion closed issue #3170: In 2.1, accumulo shell exhibits unexpected
copy/paste behavior
URL: https://github.com/apache/accumulo/issues/3170
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the sp
dlmarion commented on code in PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#discussion_r1086535003
##
core/src/main/java/org/apache/accumulo/core/client/ClientSideIteratorScanner.java:
##
@@ -213,7 +213,7 @@ public ClientSideIteratorScanner(final Scanner scanner) {
dlmarion commented on code in PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#discussion_r1086559302
##
core/src/main/java/org/apache/accumulo/core/clientImpl/TabletServerBatchReader.java:
##
@@ -116,6 +116,6 @@ public Iterator> iterator() {
}
return new
dlmarion commented on code in PR #3136:
URL: https://github.com/apache/accumulo/pull/3136#discussion_r1086632609
##
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##
@@ -1058,45 +1044,6 @@ public enum Property {
+ "constraint.",
"2.0.0"),
-
dlmarion commented on code in PR #3136:
URL: https://github.com/apache/accumulo/pull/3136#discussion_r1086632609
##
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##
@@ -1058,45 +1044,6 @@ public enum Property {
+ "constraint.",
"2.0.0"),
-
dlmarion commented on code in PR #3136:
URL: https://github.com/apache/accumulo/pull/3136#discussion_r1086724221
##
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##
@@ -1058,45 +1044,6 @@ public enum Property {
+ "constraint.",
"2.0.0"),
-
EdColeman commented on code in PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#discussion_r1086735011
##
core/src/main/java/org/apache/accumulo/core/client/ClientSideIteratorScanner.java:
##
@@ -213,7 +213,7 @@ public ClientSideIteratorScanner(final Scanner scanner)
EdColeman commented on code in PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#discussion_r1086737146
##
core/src/main/java/org/apache/accumulo/core/clientImpl/TabletServerBatchReader.java:
##
@@ -116,6 +116,6 @@ public Iterator> iterator() {
}
return new
dlmarion commented on code in PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#discussion_r1086739983
##
core/src/main/java/org/apache/accumulo/core/clientImpl/TabletServerBatchReader.java:
##
@@ -116,6 +116,6 @@ public Iterator> iterator() {
}
return new
EdColeman commented on code in PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#discussion_r1086742942
##
core/src/main/java/org/apache/accumulo/core/clientImpl/TabletServerBatchReader.java:
##
@@ -116,6 +116,6 @@ public Iterator> iterator() {
}
return new
dlmarion commented on code in PR #3136:
URL: https://github.com/apache/accumulo/pull/3136#discussion_r1086742635
##
core/src/main/java/org/apache/accumulo/core/classloader/DefaultContextClassLoaderFactory.java:
##
@@ -48,55 +46,49 @@ public class DefaultContextClassLoaderFactory
ctubbsii commented on code in PR #3161:
URL: https://github.com/apache/accumulo/pull/3161#discussion_r1086869164
##
core/src/main/java/org/apache/accumulo/core/client/admin/ActiveCompaction.java:
##
@@ -119,6 +119,12 @@ public enum CompactionReason {
*/
public abstract lo
keith-ratcliffe commented on issue #3170:
URL: https://github.com/apache/accumulo/issues/3170#issuecomment-1403986210
Thanks @dlmarion !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specif
ctubbsii commented on PR #3165:
URL: https://github.com/apache/accumulo/pull/3165#issuecomment-1404020244
In trying to understand what happened, I see that I merged this forward to
the main branch for 3.0.0 from the 2.1 branch from the command-line, so GitHub
thought it was okay to delete t
dlmarion commented on code in PR #3161:
URL: https://github.com/apache/accumulo/pull/3161#discussion_r1087000387
##
core/src/main/java/org/apache/accumulo/core/client/admin/ActiveCompaction.java:
##
@@ -119,6 +119,12 @@ public enum CompactionReason {
*/
public abstract lo
dlmarion commented on PR #3161:
URL: https://github.com/apache/accumulo/pull/3161#issuecomment-1404032242
@keith-turner - could you take a look at this when you get a chance? I
talked with @ctubbsii and he said he had not objection to including in 3.0,
wanted to get your input before I merg
EdColeman commented on PR #3165:
URL: https://github.com/apache/accumulo/pull/3165#issuecomment-1404036393
I was incorrectly using my clone as a "staging branch" instead of pushing
directly to the main (upstream) accumulo repository, thinking that would be a
safer course. The githib action
ctubbsii closed issue #64: Update docker config to work with 2.1
URL: https://github.com/apache/accumulo-proxy/issues/64
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsu
ctubbsii merged PR #65:
URL: https://github.com/apache/accumulo-proxy/pull/65
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@
ctubbsii commented on PR #3166:
URL: https://github.com/apache/accumulo/pull/3166#issuecomment-1404054373
Superseded by #3172
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment
ctubbsii commented on PR #3165:
URL: https://github.com/apache/accumulo/pull/3165#issuecomment-1404061200
You can still stage on your clone to run GitHub Actions in your fork... and
then fix any issues by adding fixes to your local branch before pushing up. You
don't need to create a PR to
ddanielr opened a new pull request, #3173:
URL: https://github.com/apache/accumulo/pull/3173
I'm looking for feedback on direction of thrift error handling.
I'm not seeing the errors I'm expecting, so I'm attempting to figure out
what disconnect I still have
My expectation is that
ctubbsii commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1087042300
##
server/base/src/main/java/org/apache/accumulo/server/AccumuloDataVersion.java:
##
@@ -47,18 +47,6 @@ public class AccumuloDataVersion {
*/
public static fin
EdColeman commented on PR #3172:
URL: https://github.com/apache/accumulo/pull/3172#issuecomment-1404085862
This looks like 1.10 may be a better target - but the package names have
changed - @ctubbsii do you have a preference for the way the changes are
applied? I could try an see if I can
EdColeman commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1087062854
##
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeChecker.java:
##
@@ -0,0 +1,113 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
dlmarion commented on code in PR #3173:
URL: https://github.com/apache/accumulo/pull/3173#discussion_r1087066114
##
server/manager/src/main/java/org/apache/accumulo/manager/ManagerClientServiceHandler.java:
##
@@ -434,7 +441,7 @@ public void setSystemProperty(TInfo info, TCreden
ctubbsii commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1087125107
##
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeChecker.java:
##
@@ -0,0 +1,113 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF)
ctubbsii commented on code in PR #3173:
URL: https://github.com/apache/accumulo/pull/3173#discussion_r1087162094
##
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##
@@ -1702,16 +1702,20 @@ private static boolean
hasPrefixWithAnnotation(String key
/**
ddanielr commented on code in PR #3173:
URL: https://github.com/apache/accumulo/pull/3173#discussion_r1087206989
##
server/manager/src/main/java/org/apache/accumulo/manager/ManagerClientServiceHandler.java:
##
@@ -434,7 +441,7 @@ public void setSystemProperty(TInfo info, TCreden
EdColeman commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1087252944
##
server/base/src/test/java/org/apache/accumulo/server/ServerContextTest.java:
##
@@ -135,7 +135,8 @@ public void testCanRun() {
// ensure this fails with olde
EdColeman commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1087286437
##
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeCoordinator.java:
##
@@ -142,11 +140,14 @@ public synchronized void upgradeZookeeper(Serve
EdColeman commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1087294704
##
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeChecker.java:
##
@@ -0,0 +1,113 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
EdColeman commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1087295011
##
server/base/src/main/java/org/apache/accumulo/server/AccumuloDataVersion.java:
##
@@ -47,18 +47,6 @@ public class AccumuloDataVersion {
*/
public static fi
ctubbsii commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1088008352
##
server/base/src/test/java/org/apache/accumulo/server/ServerContextTest.java:
##
@@ -135,7 +135,8 @@ public void testCanRun() {
// ensure this fails with older
ctubbsii commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1088015001
##
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeCoordinator.java:
##
@@ -142,11 +140,14 @@ public synchronized void upgradeZookeeper(Server
ctubbsii commented on code in PR #3160:
URL: https://github.com/apache/accumulo/pull/3160#discussion_r1088015001
##
server/manager/src/main/java/org/apache/accumulo/manager/upgrade/UpgradeCoordinator.java:
##
@@ -142,11 +140,14 @@ public synchronized void upgradeZookeeper(Server
ivakegg commented on PR #3142:
URL: https://github.com/apache/accumulo/pull/3142#issuecomment-1405242835
@ctubbsii Are you now ok with this PR or do you hvae additional concerns?
Do you want additional PRs to go against 2.1 and 1.10 ?
--
This is an automated message from the Apache Git S
ctubbsii commented on PR #3142:
URL: https://github.com/apache/accumulo/pull/3142#issuecomment-1405281252
@ivakegg asked:
> @ctubbsii Are you now ok with this PR or do you hvae additional concerns?
Do you want additional PRs to go against 2.1 and 1.10 ?
I don't have any other conce
ctubbsii commented on issue #1169:
URL: https://github.com/apache/accumulo/issues/1169#issuecomment-1405284059
This is addressed by #3142
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the spec
ctubbsii merged PR #3142:
URL: https://github.com/apache/accumulo/pull/3142
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@ac
asfgit closed issue #1169: balancing strategy
URL: https://github.com/apache/accumulo/issues/1169
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifi
ctubbsii commented on PR #3142:
URL: https://github.com/apache/accumulo/pull/3142#issuecomment-1405328601
Committed to 2.1 branch as 7f19f96914ed12614bbd2d6b24a88d322e49b006 and
merged forward into main branch
--
This is an automated message from the Apache Git Service.
To respond to the
ctubbsii commented on code in PR #3167:
URL: https://github.com/apache/accumulo/pull/3167#discussion_r1088130572
##
core/src/main/java/org/apache/accumulo/core/fate/Fate.java:
##
@@ -235,6 +235,9 @@ public Fate(T environment, TStore store,
Function,String> toLogStrFun
* Lau
DomGarguilo opened a new pull request, #67:
URL: https://github.com/apache/accumulo-proxy/pull/67
Fixes #66
The proxy now looks for the property `sharedSecret` in the props file. Also,
the changes in #59 removed the login method.
This PR fixes these two problems.
--
This is
DomGarguilo commented on PR #67:
URL: https://github.com/apache/accumulo-proxy/pull/67#issuecomment-1405397740
Before adding `sharedSecret` to the example properties file, it was being
set to `null`. I am wondering if there should be a check in place to make sure
that the shared secret is n
ivakegg commented on PR #3142:
URL: https://github.com/apache/accumulo/pull/3142#issuecomment-1405487315
@ctubbsii OK, thank you for merging. I can keep 1.10 backport exclusively
in our own repo.
--
This is an automated message from the Apache Git Service.
To respond to the message, plea
ddanielr commented on code in PR #3173:
URL: https://github.com/apache/accumulo/pull/3173#discussion_r1088250767
##
core/src/main/java/org/apache/accumulo/core/conf/Property.java:
##
@@ -1702,16 +1702,20 @@ private static boolean
hasPrefixWithAnnotation(String key
/**
dlmarion opened a new issue, #3174:
URL: https://github.com/apache/accumulo/issues/3174
There is some confusing logic/use of this static map that I don't
understand. In the places that it's updated, servers are only added to it,
never removed. In the way that it's used, it sets the boolean
jmark99 opened a new pull request, #115:
URL: https://github.com/apache/accumulo-examples/pull/115
Provide step by step example of how to use the uniquecols example for the
2.1 examples repository.
--
This is an automated message from the Apache Git Service.
To respond to the message, ple
jmark99 merged PR #115:
URL: https://github.com/apache/accumulo-examples/pull/115
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr
DomGarguilo opened a new pull request, #68:
URL: https://github.com/apache/accumulo-proxy/pull/68
This PR:
* removes tests or portions of tests that switch proxy users
* misc. improvements to other test logic, names or logging
--
This is an automated message from the Apache Git Servi
ddanielr commented on code in PR #3146:
URL: https://github.com/apache/accumulo/pull/3146#discussion_r1088690055
##
shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java:
##
@@ -353,6 +344,31 @@ public int execute(final String fullCommand, final
CommandLine
ddanielr commented on code in PR #3146:
URL: https://github.com/apache/accumulo/pull/3146#discussion_r1088691250
##
shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java:
##
@@ -353,6 +344,31 @@ public int execute(final String fullCommand, final
CommandLine
EdColeman commented on code in PR #3146:
URL: https://github.com/apache/accumulo/pull/3146#discussion_r1088990088
##
shell/src/main/java/org/apache/accumulo/shell/commands/ConfigCommand.java:
##
@@ -353,6 +344,31 @@ public int execute(final String fullCommand, final
CommandLine
1 - 100 of 12286 matches
Mail list logo