[GitHub] [accumulo] ctubbsii commented on issue #2820: Additional improvements to the du command,

2022-09-01 Thread GitBox
ctubbsii commented on issue #2820: URL: https://github.com/apache/accumulo/issues/2820#issuecomment-1234906215 See my comments on the PR. -- 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

[GitHub] [accumulo] ctubbsii commented on a diff in pull request #2900: Add a new "dumeta" command to compute table disk usage

2022-09-01 Thread GitBox
ctubbsii commented on code in PR #2900: URL: https://github.com/apache/accumulo/pull/2900#discussion_r961154438 ## core/src/main/java/org/apache/accumulo/core/client/admin/TableOperations.java: ## @@ -1071,6 +1071,24 @@ Map listConstraints(String tableName) List

[GitHub] [accumulo] ctubbsii opened a new pull request, #2912: Updates to ScanAttempts

2022-09-01 Thread GitBox
ctubbsii opened a new pull request, #2912: URL: https://github.com/apache/accumulo/pull/2912 As a follow-up to #2880: * Remove unused and undocumented endTime in ScanAttempt API * Rename ScanAttemptImpl method and variables from "mutationCounter" to "attemptNumber" with

[GitHub] [accumulo] ctubbsii merged pull request #2909: Mark FileSystemMonitor as deprecated and slated for 3.0 removal

2022-09-01 Thread GitBox
ctubbsii merged PR #2909: URL: https://github.com/apache/accumulo/pull/2909 -- 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:

[GitHub] [accumulo] jmark99 commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
jmark99 commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r961077160 ## core/src/main/java/org/apache/accumulo/core/clientImpl/TableOperationsImpl.java: ## @@ -2011,6 +2015,34 @@ public ImportDestinationArguments importDirectory(String

[GitHub] [accumulo] jmark99 commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
jmark99 commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r961075476 ## test/src/main/java/org/apache/accumulo/test/TableOperationsIT.java: ## @@ -292,4 +295,59 @@ public void testCompactEmptyTablesWithBadIterator_FailsAndCancel()

[GitHub] [accumulo] jmark99 commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
jmark99 commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r961074258 ## test/src/main/java/org/apache/accumulo/test/TableOperationsIT.java: ## @@ -292,4 +295,59 @@ public void testCompactEmptyTablesWithBadIterator_FailsAndCancel()

[GitHub] [accumulo] jmark99 commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
jmark99 commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r961073453 ## core/src/main/java/org/apache/accumulo/core/clientImpl/TableOperationsImpl.java: ## @@ -2011,6 +2015,34 @@ public ImportDestinationArguments importDirectory(String

[GitHub] [accumulo] dlmarion commented on a diff in pull request #2197: Per table crypto + other crypto improvements

2022-09-01 Thread GitBox
dlmarion commented on code in PR #2197: URL: https://github.com/apache/accumulo/pull/2197#discussion_r960956356 ## test/src/main/java/org/apache/accumulo/test/performance/scan/CollectTabletStats.java: ## @@ -464,7 +464,7 @@ private static int readFiles(VolumeManager fs,

[GitHub] [accumulo] ctubbsii merged pull request #2911: Serialize location in BadLocationStateException

2022-09-01 Thread GitBox
ctubbsii merged PR #2911: URL: https://github.com/apache/accumulo/pull/2911 -- 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:

[GitHub] [accumulo-website] jmark99 merged pull request #334: Replace older Tour with newer JShell Tour

2022-09-01 Thread GitBox
jmark99 merged PR #334: URL: https://github.com/apache/accumulo-website/pull/334 -- 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:

[GitHub] [accumulo] ctubbsii commented on pull request #2906: convert text to printable string before logging in merge tablets

2022-09-01 Thread GitBox
ctubbsii commented on PR #2906: URL: https://github.com/apache/accumulo/pull/2906#issuecomment-1234611631 Thanks for the PR @skirklin . It looks like this is your first contribution to this project. If you wish to be added as a contributor to https://accumulo.apache.org/people/ , please

[GitHub] [accumulo] ctubbsii commented on a diff in pull request #2897: Make Text field transient in BadLocationStateException

2022-09-01 Thread GitBox
ctubbsii commented on code in PR #2897: URL: https://github.com/apache/accumulo/pull/2897#discussion_r960955209 ## core/src/main/java/org/apache/accumulo/core/metadata/TabletLocationState.java: ## @@ -36,7 +36,7 @@ public class TabletLocationState { public static class

[GitHub] [accumulo] ctubbsii opened a new pull request, #2911: Serialize location in BadLocationStateException

2022-09-01 Thread GitBox
ctubbsii opened a new pull request, #2911: URL: https://github.com/apache/accumulo/pull/2911 To ensure that the location is correctly serialized in this exception, do not mark it transient, but convert it to a byte array, since Text is not Serializable. This addresses the

[GitHub] [accumulo-website] jmark99 commented on a diff in pull request #334: Replace older Tour with newer JShell Tour

2022-09-01 Thread GitBox
jmark99 commented on code in PR #334: URL: https://github.com/apache/accumulo-website/pull/334#discussion_r960952152 ## tour/batch-scanner-code.md: ## @@ -4,44 +4,52 @@ title: Batch Scanner Code Below is a solution to the exercise. -```java - static void

[GitHub] [accumulo] ctubbsii commented on a diff in pull request #2897: Make Text field transient in BadLocationStateException

2022-09-01 Thread GitBox
ctubbsii commented on code in PR #2897: URL: https://github.com/apache/accumulo/pull/2897#discussion_r960931792 ## core/src/main/java/org/apache/accumulo/core/metadata/TabletLocationState.java: ## @@ -36,7 +36,7 @@ public class TabletLocationState { public static class

[GitHub] [accumulo] ctubbsii commented on issue #2908: MiniAccumuloCluster binds admin.serverPort to 8080

2022-09-01 Thread GitBox
ctubbsii commented on issue #2908: URL: https://github.com/apache/accumulo/issues/2908#issuecomment-1234575819 > Except, duh-uh, of course setting the system property doesn't work - because MAC spawns ZK in a separate JVM process. Sigh ... I'll have to resort to one of the uglier

[GitHub] [accumulo] DomGarguilo closed issue #2905: Merging tablets via mergomatic prints unformated Text to terminal

2022-09-01 Thread GitBox
DomGarguilo closed issue #2905: Merging tablets via mergomatic prints unformated Text to terminal URL: https://github.com/apache/accumulo/issues/2905 -- 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

[GitHub] [accumulo] DomGarguilo merged pull request #2906: convert text to printable string before logging in merge tablets

2022-09-01 Thread GitBox
DomGarguilo merged PR #2906: URL: https://github.com/apache/accumulo/pull/2906 -- 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:

[GitHub] [accumulo] keith-turner commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
keith-turner commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r960898970 ## core/src/main/java/org/apache/accumulo/core/clientImpl/TableOperationsImpl.java: ## @@ -2011,6 +2015,34 @@ public ImportDestinationArguments

[GitHub] [accumulo] keith-turner commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
keith-turner commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r960898970 ## core/src/main/java/org/apache/accumulo/core/clientImpl/TableOperationsImpl.java: ## @@ -2011,6 +2015,34 @@ public ImportDestinationArguments

[GitHub] [accumulo] keith-turner commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
keith-turner commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r960898970 ## core/src/main/java/org/apache/accumulo/core/clientImpl/TableOperationsImpl.java: ## @@ -2011,6 +2015,34 @@ public ImportDestinationArguments

[GitHub] [accumulo] DomGarguilo commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
DomGarguilo commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r960850160 ## test/src/main/java/org/apache/accumulo/test/TableOperationsIT.java: ## @@ -292,4 +295,59 @@ public void testCompactEmptyTablesWithBadIterator_FailsAndCancel()

[GitHub] [accumulo] milleruntime merged pull request #2904: Fix Fate operation name

2022-09-01 Thread GitBox
milleruntime merged PR #2904: URL: https://github.com/apache/accumulo/pull/2904 -- 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:

[GitHub] [accumulo] dlmarion commented on a diff in pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
dlmarion commented on code in PR #2910: URL: https://github.com/apache/accumulo/pull/2910#discussion_r960780534 ## core/src/main/java/org/apache/accumulo/core/clientImpl/TableOperationsImpl.java: ## @@ -2011,6 +2015,34 @@ public ImportDestinationArguments importDirectory(String

[GitHub] [accumulo] jmark99 commented on pull request #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
jmark99 commented on PR #2910: URL: https://github.com/apache/accumulo/pull/2910#issuecomment-1234413246 Ticket replaces Jira issue https://issues.apache.org/jira/browse/ACCUMULO-3447 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[jira] [Resolved] (ACCUMULO-3447) No way to programmatically get table's TimeType

2022-09-01 Thread Mark Owens (Jira)
[ https://issues.apache.org/jira/browse/ACCUMULO-3447?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Mark Owens resolved ACCUMULO-3447. -- Resolution: Done Closing ticket as it is replaced by Accumulo GitHub issue #2910 (Add

[GitHub] [accumulo] jmark99 opened a new pull request, #2910: Add ability to retrieve TimeType for a table

2022-09-01 Thread GitBox
jmark99 opened a new pull request, #2910: URL: https://github.com/apache/accumulo/pull/2910 Provide a way to programmatically retrieve a table's TimeType value. Added client method 'gettimetype' and corresponding shell command that retrieves the TimeType for a supplied table. --

[GitHub] [accumulo] EdColeman opened a new pull request, #2909: Mark FileSystemMonitor as deprecated and slated for 3.0 removal

2022-09-01 Thread GitBox
EdColeman opened a new pull request, #2909: URL: https://github.com/apache/accumulo/pull/2909 For 3.0, PR #1328 will remove the FileSystemMonitor that monitors the mounted file systems and can kill the server if a disk goes read-only. With WALs using hdfs, the primary driver for this