[GitHub] [accumulo] ctubbsii opened a new issue #1574: Periodic failure of MetadataMaxFilesIT

2020-03-27 Thread GitBox
ctubbsii opened a new issue #1574: Periodic failure of MetadataMaxFilesIT URL: https://github.com/apache/accumulo/issues/1574 The description of this test failure (involving a split and unresponsive tserver) has elements in common with what @keith-turner described on #1076, which also had

[GitHub] [accumulo-website] arvindshmicrosoft opened a new pull request #225: Add instructions for running multiple TServers per node in Accumulo 2.1.0

2020-03-27 Thread GitBox
arvindshmicrosoft opened a new pull request #225: Add instructions for running multiple TServers per node in Accumulo 2.1.0 URL: https://github.com/apache/accumulo-website/pull/225 * Explains the usage of accumulo-cluster and NUM_TSERVERS * Adds details of filenames when running more

[GitHub] [accumulo-testing] billierinaldi commented on issue #123: Create a tool for verifying WAL assumptions

2020-03-27 Thread GitBox
billierinaldi commented on issue #123: Create a tool for verifying WAL assumptions URL: https://github.com/apache/accumulo-testing/pull/123#issuecomment-605380233 Thanks for taking a look, @ctubbsii and @keith-turner. That's a good point. One option would be for this to be a standalone

[GitHub] [accumulo-testing] keith-turner edited a comment on issue #123: Create a tool for verifying WAL assumptions

2020-03-27 Thread GitBox
keith-turner edited a comment on issue #123: Create a tool for verifying WAL assumptions URL: https://github.com/apache/accumulo-testing/pull/123#issuecomment-605340518 Looking at the test its use of internal accumulo config and volume manager are likely susceptible to breakage between

[GitHub] [accumulo-testing] keith-turner commented on issue #123: Create a tool for verifying WAL assumptions

2020-03-27 Thread GitBox
keith-turner commented on issue #123: Create a tool for verifying WAL assumptions URL: https://github.com/apache/accumulo-testing/pull/123#issuecomment-605340518 Looking at the test its use of internal accumulo config and volume manager are likely susceptible to breakage between accumulo

[GitHub] [accumulo] keith-turner commented on a change in pull request #1568: Support multiple tservers / node in accumulo-service

2020-03-27 Thread GitBox
keith-turner commented on a change in pull request #1568: Support multiple tservers / node in accumulo-service URL: https://github.com/apache/accumulo/pull/1568#discussion_r399562448 ## File path: assemble/bin/accumulo-cluster ## @@ -100,15 +100,45 @@ function get_ip() {

[GitHub] [accumulo] milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399547294 ## File path: server/base/src/main/java/org/apache/accumulo/server/ServerContext.java ## @@ -112,7

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
ctubbsii commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399542219 ## File path: server/base/src/main/java/org/apache/accumulo/server/ServerContext.java ## @@ -112,7

[GitHub] [accumulo] ctubbsii merged pull request #1568: Support multiple tservers / node in accumulo-service

2020-03-27 Thread GitBox
ctubbsii merged pull request #1568: Support multiple tservers / node in accumulo-service URL: https://github.com/apache/accumulo/pull/1568 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [accumulo] ctubbsii commented on issue #1568: Support multiple tservers / node in accumulo-service

2020-03-27 Thread GitBox
ctubbsii commented on issue #1568: Support multiple tservers / node in accumulo-service URL: https://github.com/apache/accumulo/pull/1568#issuecomment-605311214 I tested this by setting `NUM_TSERVERS=3` in my Uno config file. I ran into issue #1573, but worked around it with:

[GitHub] [accumulo] ctubbsii opened a new issue #1573: tserver.port.search=true steals master port

2020-03-27 Thread GitBox
ctubbsii opened a new issue #1573: tserver.port.search=true steals master port URL: https://github.com/apache/accumulo/issues/1573 I tried to start 3 tservers, with the following config: ```properties ; default ports master.port.client= tserver.port.client=9997 ;

[GitHub] [accumulo] milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399470557 ## File path: server/base/src/main/java/org/apache/accumulo/server/tabletserver/MemoryManager.java

[GitHub] [accumulo] milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399469259 ## File path: server/base/src/main/java/org/apache/accumulo/server/ServerContext.java ## @@ -112,7

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
ctubbsii commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399459647 ## File path: server/base/src/main/java/org/apache/accumulo/server/tabletserver/MemoryManager.java ##

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
ctubbsii commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399458608 ## File path: server/base/src/main/java/org/apache/accumulo/server/tabletserver/MemoryManager.java ##

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
ctubbsii commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399449457 ## File path: server/base/src/main/java/org/apache/accumulo/server/ServerContext.java ## @@ -112,7

[GitHub] [accumulo] milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399455150 ## File path:

[GitHub] [accumulo] milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399454328 ## File path:

[GitHub] [accumulo] cradal commented on a change in pull request #1563: Work-In-Progress Fixes #1473 - Move TabletsMetaData builder to Ample

2020-03-27 Thread GitBox
cradal commented on a change in pull request #1563: Work-In-Progress Fixes #1473 - Move TabletsMetaData builder to Ample URL: https://github.com/apache/accumulo/pull/1563#discussion_r399396815 ## File path: core/src/main/java/org/apache/accumulo/core/metadata/schema/AmpleImpl.java

[GitHub] [accumulo] cradal commented on a change in pull request #1563: Work-In-Progress Fixes #1473 - Move TabletsMetaData builder to Ample

2020-03-27 Thread GitBox
cradal commented on a change in pull request #1563: Work-In-Progress Fixes #1473 - Move TabletsMetaData builder to Ample URL: https://github.com/apache/accumulo/pull/1563#discussion_r399389467 ## File path: core/src/main/java/org/apache/accumulo/core/metadata/schema/AmpleImpl.java

[GitHub] [accumulo] arvindshmicrosoft commented on issue #1568: Support multiple tservers / node in accumulo-service

2020-03-27 Thread GitBox
arvindshmicrosoft commented on issue #1568: Support multiple tservers / node in accumulo-service URL: https://github.com/apache/accumulo/pull/1568#issuecomment-605033055 Thank you @ctubbsii for the guidance and insights. I learnt a lot in this process. I'll follow up with a PR to update

[GitHub] [accumulo] milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory

2020-03-27 Thread GitBox
milleruntime commented on a change in pull request #1572: Refactor use of ServerConfigurationFactory URL: https://github.com/apache/accumulo/pull/1572#discussion_r399252881 ## File path:

[GitHub] [accumulo] ctubbsii commented on issue #1568: Support multiple tservers / node in accumulo-service

2020-03-27 Thread GitBox
ctubbsii commented on issue #1568: Support multiple tservers / node in accumulo-service URL: https://github.com/apache/accumulo/pull/1568#issuecomment-604908642 Thanks @arvindshmicrosoft ! I really like how this change worked out. This is a nice feature to have in our cluster management

[GitHub] [accumulo] ctubbsii edited a comment on issue #1568: Support multiple tservers / node in accumulo-service

2020-03-27 Thread GitBox
ctubbsii edited a comment on issue #1568: Support multiple tservers / node in accumulo-service URL: https://github.com/apache/accumulo/pull/1568#issuecomment-604908642 Thanks @arvindshmicrosoft ! I really like how this change worked out. This is a nice feature to have in our out-of-box

[GitHub] [accumulo] arvindshmicrosoft commented on a change in pull request #1568: Support multiple tservers / node in accumulo-service

2020-03-27 Thread GitBox
arvindshmicrosoft commented on a change in pull request #1568: Support multiple tservers / node in accumulo-service URL: https://github.com/apache/accumulo/pull/1568#discussion_r399076381 ## File path: assemble/bin/accumulo-cluster ## @@ -100,15 +100,45 @@ function

[GitHub] [accumulo] arvindshmicrosoft commented on a change in pull request #1568: Support multiple tservers / node in accumulo-service

2020-03-27 Thread GitBox
arvindshmicrosoft commented on a change in pull request #1568: Support multiple tservers / node in accumulo-service URL: https://github.com/apache/accumulo/pull/1568#discussion_r399075699 ## File path: assemble/bin/accumulo-cluster ## @@ -100,15 +100,45 @@ function