(accumulo) branch 2.1 updated: Add scan server prefix to valid zooProp list (#4487)
This is an automated email from the ASF dual-hosted git repository.
ddanielr pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/2.1 by this push:
new 0958a5a456 Add scan server prefix to valid zooProp list (#4487)
0958a5a456 is described below
commit 0958a5a4565900a487b7a5ee260ae6099ab8548e
Author: Daniel Roberts
AuthorDate: Tue Apr 23 13:23:58 2024 -0400
Add scan server prefix to valid zooProp list (#4487)
* Add scan server prefix to valid zooProp list
Adds the scan server properties prefix to the list of valid zooProps.
This allows the scan server properties to be modified via the shell.
* Updates fixed properties
Updates the list of fixed properties to include properties that are used
when setting up a server and are never checked again at runtime.
* moved sserver prop
---
.../org/apache/accumulo/core/conf/Property.java| 24 +++---
1 file changed, 21 insertions(+), 3 deletions(-)
diff --git a/core/src/main/java/org/apache/accumulo/core/conf/Property.java
b/core/src/main/java/org/apache/accumulo/core/conf/Property.java
index 1db7d9326d..e6b7707235 100644
--- a/core/src/main/java/org/apache/accumulo/core/conf/Property.java
+++ b/core/src/main/java/org/apache/accumulo/core/conf/Property.java
@@ -1832,11 +1832,28 @@ public enum Property {
public static final EnumSet fixedProperties = EnumSet.of(
// port options
- GC_PORT, MANAGER_CLIENTPORT, TSERV_CLIENTPORT,
+ GC_PORT, MANAGER_CLIENTPORT, TSERV_CLIENTPORT, SSERV_CLIENTPORT,
SSERV_PORTSEARCH,
+ COMPACTOR_PORTSEARCH, TSERV_PORTSEARCH,
- // tserver cache options
+ // max message options
+ SSERV_MAX_MESSAGE_SIZE, TSERV_MAX_MESSAGE_SIZE,
COMPACTOR_MAX_MESSAGE_SIZE,
+ COMPACTION_COORDINATOR_MAX_MESSAGE_SIZE,
+
+ // block cache options
TSERV_CACHE_MANAGER_IMPL, TSERV_DATACACHE_SIZE, TSERV_INDEXCACHE_SIZE,
- TSERV_SUMMARYCACHE_SIZE,
+ TSERV_SUMMARYCACHE_SIZE, SSERV_DATACACHE_SIZE, SSERV_INDEXCACHE_SIZE,
SSERV_SUMMARYCACHE_SIZE,
+
+ // blocksize options
+ TSERV_DEFAULT_BLOCKSIZE, SSERV_DEFAULT_BLOCKSIZE,
+
+ // sserver specific options
+ SSERVER_SCAN_REFERENCE_EXPIRATION_TIME,
SSERV_CACHED_TABLET_METADATA_EXPIRATION,
+
+ // thread options
+ TSERV_MINTHREADS, TSERV_MINTHREADS_TIMEOUT, SSERV_MINTHREADS,
SSERV_MINTHREADS_TIMEOUT,
+ COMPACTION_COORDINATOR_MINTHREADS,
COMPACTION_COORDINATOR_MINTHREADS_TIMEOUT,
+ MANAGER_MINTHREADS, MANAGER_MINTHREADS_TIMEOUT, COMPACTOR_MINTHREADS,
+ COMPACTOR_MINTHREADS_TIMEOUT,
// others
TSERV_NATIVEMAP_ENABLED, TSERV_SCAN_MAX_OPENFILES,
MANAGER_RECOVERY_WAL_EXISTENCE_CACHE_TIME);
@@ -1862,6 +1879,7 @@ public enum Property {
// white list prefixes
return key.startsWith(Property.TABLE_PREFIX.getKey())
|| key.startsWith(Property.TSERV_PREFIX.getKey())
+|| key.startsWith(Property.SSERV_PREFIX.getKey())
|| key.startsWith(Property.MANAGER_PREFIX.getKey())
|| key.startsWith(Property.MASTER_PREFIX.getKey())
|| key.startsWith(Property.GC_PREFIX.getKey())
(accumulo-website) branch asf-site updated: Automatic Site Publish by Buildbot
This is an automated email from the ASF dual-hosted git repository. git-site-role pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/accumulo-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 25d550f8 Automatic Site Publish by Buildbot 25d550f8 is described below commit 25d550f80201ed502af6a8239c123c0750cc5468 Author: buildbot AuthorDate: Tue Apr 23 13:43:05 2024 + Automatic Site Publish by Buildbot --- output/Dockerfile | 2 +- output/feed.xml | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/output/Dockerfile b/output/Dockerfile index 3554c0bf..2798588a 100644 --- a/output/Dockerfile +++ b/output/Dockerfile @@ -31,4 +31,4 @@ ENV PORT=4000 EXPOSE $PORT # Configure the default command to build from the mounted repository. -CMD bundle exec jekyll serve --force-polling -H $HOST -P $PORT +CMD bundle exec jekyll serve -H $HOST -P $PORT diff --git a/output/feed.xml b/output/feed.xml index bbc7ab9b..5f690675 100644 --- a/output/feed.xml +++ b/output/feed.xml @@ -6,8 +6,8 @@ https://accumulo.apache.org/ https://accumulo.apache.org/feed.xml; rel="self" type="application/rss+xml"/> -Mon, 22 Apr 2024 18:30:52 + -Mon, 22 Apr 2024 18:30:52 + +Tue, 23 Apr 2024 13:42:58 + +Tue, 23 Apr 2024 13:42:58 + Jekyll v4.3.2
(accumulo-website) branch main updated: Small improvements to development dockerfile (#422)
This is an automated email from the ASF dual-hosted git repository. domgarguilo pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/accumulo-website.git The following commit(s) were added to refs/heads/main by this push: new 099673a5 Small improvements to development dockerfile (#422) 099673a5 is described below commit 099673a5aefcc49474215f16df2a616660d1157a Author: Dom G AuthorDate: Tue Apr 23 09:42:38 2024 -0400 Small improvements to development dockerfile (#422) * remove --force-polling from jekyll command * remove exclude tag from config file for simplicity --- Dockerfile | 2 +- _config.yml | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/Dockerfile b/Dockerfile index 3554c0bf..2798588a 100644 --- a/Dockerfile +++ b/Dockerfile @@ -31,4 +31,4 @@ ENV PORT=4000 EXPOSE $PORT # Configure the default command to build from the mounted repository. -CMD bundle exec jekyll serve --force-polling -H $HOST -P $PORT +CMD bundle exec jekyll serve -H $HOST -P $PORT diff --git a/_config.yml b/_config.yml index 845d17e9..5baea8d5 100644 --- a/_config.yml +++ b/_config.yml @@ -13,7 +13,6 @@ description: > # this means to ignore newlines until "url:" The Apache Accumulo™ sorted, distributed key/value store is a robust, scalable, high performance data storage and retrieval system. url: "https://accumulo.apache.org; # the base hostname & protocol for your site -exclude: [vendor] latest_release: 2.1.2 javadoc_version: 2.1.2 num_home_posts: 5
(accumulo) branch elasticity updated: Fixed check in TabletResourceGroupBalanceIT to use Wait.waitFor (#4479)
This is an automated email from the ASF dual-hosted git repository.
dlmarion pushed a commit to branch elasticity
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/elasticity by this push:
new ee04ad3aa5 Fixed check in TabletResourceGroupBalanceIT to use
Wait.waitFor (#4479)
ee04ad3aa5 is described below
commit ee04ad3aa522404cb37a1eceefd2bec34b2d00e7
Author: Dave Marion
AuthorDate: Tue Apr 23 07:28:56 2024 -0400
Fixed check in TabletResourceGroupBalanceIT to use Wait.waitFor (#4479)
The assertion in testResourceGroupBalanceWithNoTServers started returning
zero for the number of hosted tablets after the `waitForBalance`. Not sure
which modification caused this to change behavior, but this fix is likely
the correct one regardless.
---
.../functional/TabletResourceGroupBalanceIT.java | 30 --
1 file changed, 16 insertions(+), 14 deletions(-)
diff --git
a/test/src/main/java/org/apache/accumulo/test/functional/TabletResourceGroupBalanceIT.java
b/test/src/main/java/org/apache/accumulo/test/functional/TabletResourceGroupBalanceIT.java
index 6b289a72b4..0605a69c2a 100644
---
a/test/src/main/java/org/apache/accumulo/test/functional/TabletResourceGroupBalanceIT.java
+++
b/test/src/main/java/org/apache/accumulo/test/functional/TabletResourceGroupBalanceIT.java
@@ -225,20 +225,22 @@ public class TabletResourceGroupBalanceIT extends
SharedMiniClusterBase {
.addTabletServerResourceGroup("GROUP2", 1);
getCluster().getClusterControl().start(ServerType.TABLET_SERVER);
- client.instanceOperations().waitForBalance();
- assertEquals(26, getCountOfHostedTablets(client, tableName));
- ingest.join();
- assertNull(error.get());
-
- client.tableOperations().delete(tableName);
- // Stop all tablet servers because there is no way to just stop
- // the GROUP2 server yet.
-
getCluster().getClusterControl().stopAllServers(ServerType.TABLET_SERVER);
-
getCluster().getConfig().getClusterServerConfiguration().clearTServerResourceGroups();
- getCluster().getConfig().getClusterServerConfiguration()
- .addTabletServerResourceGroup("GROUP1", 1);
- getCluster().getClusterControl().start(ServerType.TABLET_SERVER);
+ try {
+client.instanceOperations().waitForBalance();
+Wait.waitFor(() -> getCountOfHostedTablets(client, tableName) == 26);
+ingest.join();
+assertNull(error.get());
+ } finally {
+client.tableOperations().delete(tableName);
+// Stop all tablet servers because there is no way to just stop
+// the GROUP2 server yet.
+
getCluster().getClusterControl().stopAllServers(ServerType.TABLET_SERVER);
+
getCluster().getConfig().getClusterServerConfiguration().clearTServerResourceGroups();
+getCluster().getConfig().getClusterServerConfiguration()
+.addTabletServerResourceGroup("GROUP1", 1);
+getCluster().getClusterControl().start(ServerType.TABLET_SERVER);
+ }
}
}
@@ -258,7 +260,7 @@ public class TabletResourceGroupBalanceIT extends
SharedMiniClusterBase {
client.tableOperations().create(tableName, ntc1);
// wait for all tablets to be hosted
- Wait.waitFor(() -> 26 != getCountOfHostedTablets(client, tableName));
+ Wait.waitFor(() -> 26 == getCountOfHostedTablets(client, tableName));
client.instanceOperations().waitForBalance();
(accumulo) branch elasticity updated: Allow tablet refresh while in the process of closing (#4483)
This is an automated email from the ASF dual-hosted git repository.
dlmarion pushed a commit to branch elasticity
in repository https://gitbox.apache.org/repos/asf/accumulo.git
The following commit(s) were added to refs/heads/elasticity by this push:
new 45653be366 Allow tablet refresh while in the process of closing (#4483)
45653be366 is described below
commit 45653be36678c9f4fa970e7db819ad67c6b02113
Author: Keith Turner
AuthorDate: Tue Apr 23 07:27:47 2024 -0400
Allow tablet refresh while in the process of closing (#4483)
There was check in the tablet refresh code that was preventing tablet
refresh while a tablet was in the middle of closing. Modified the check
to only prevent refresh after a tablet is competely closed.
fixes #4477
---
.../src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java| 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
index 348807f161..38cd3cff18 100644
---
a/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
+++
b/server/tserver/src/main/java/org/apache/accumulo/tserver/tablet/Tablet.java
@@ -1635,7 +1635,7 @@ public class Tablet extends TabletBase {
}
synchronized (this) {
-if (isClosed()) {
+if (isCloseComplete()) {
log.debug("Unable to refresh tablet {} for {} because the tablet is
closed", extent,
refreshPurpose);
return false;
