(accumulo) 01/01: Merge branch '2.1'

2024-05-01 Thread kturner
This is an automated email from the ASF dual-hosted git repository.

kturner pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git

commit 8da73ce467f9311d247ac1eaa8062a5f62853276
Merge: 09adb11491 61ca7d8070
Author: Keith Turner 
AuthorDate: Wed May 1 22:59:21 2024 +

Merge branch '2.1'

 .../coordinator/CompactionCoordinator.java |  8 +++--
 .../coordinator/DeadCompactionDetector.java| 11 +--
 .../accumulo/manager/TabletGroupWatcher.java   | 35 ++
 3 files changed, 38 insertions(+), 16 deletions(-)




(accumulo) branch main updated (09adb11491 -> 8da73ce467)

2024-05-01 Thread kturner
This is an automated email from the ASF dual-hosted git repository.

kturner pushed a change to branch main
in repository https://gitbox.apache.org/repos/asf/accumulo.git


from 09adb11491 enable metrics log to file for ExternalCompactionMetricsIT 
(#4502)
 add 2ce89e0ec4 uses specific logger for compaction coordinator status 
update (#4489)
 add acaa6dc46c lowers logging level for first seen dead compaction (#4490)
 add 61ca7d8070 Handles RPC errors when requesting tablet unload (#4497)
 new 8da73ce467 Merge branch '2.1'

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../coordinator/CompactionCoordinator.java |  8 +++--
 .../coordinator/DeadCompactionDetector.java| 11 +--
 .../accumulo/manager/TabletGroupWatcher.java   | 35 ++
 3 files changed, 38 insertions(+), 16 deletions(-)



(accumulo) branch 2.1 updated: Handles RPC errors when requesting tablet unload (#4497)

2024-05-01 Thread kturner
This is an automated email from the ASF dual-hosted git repository.

kturner 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 61ca7d8070 Handles RPC errors when requesting tablet unload (#4497)
61ca7d8070 is described below

commit 61ca7d8070f2eedabf987f5fb471b475babe7da5
Author: Keith Turner 
AuthorDate: Wed May 1 18:55:51 2024 -0400

Handles RPC errors when requesting tablet unload (#4497)

The tablet server group watcher loop will not make progress when it 
encounters
an RPC error on a single tablet server.  It should continue communicating 
with
the tablet servers it can inorder to make progress in its loop that assigns
and unassigns tablets.
---
 .../accumulo/manager/TabletGroupWatcher.java   | 35 ++
 1 file changed, 23 insertions(+), 12 deletions(-)

diff --git 
a/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
 
b/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
index 8171aa..216526d328 100644
--- 
a/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
+++ 
b/server/manager/src/main/java/org/apache/accumulo/manager/TabletGroupWatcher.java
@@ -344,12 +344,17 @@ abstract class TabletGroupWatcher extends 
AccumuloDaemonThread {
 TServerConnection client =
 
manager.tserverSet.getConnection(location.getServerInstance());
 if (client != null) {
-  Manager.log.trace("[{}] Requesting TabletServer {} unload {} 
{}", store.name(),
-  location.getServerInstance(), tls.extent, 
goal.howUnload());
-  client.unloadTablet(manager.managerLock, tls.extent, 
goal.howUnload(),
-  manager.getSteadyTime());
-  unloaded++;
-  totalUnloaded++;
+  try {
+Manager.log.trace("[{}] Requesting TabletServer {} unload 
{} {}", store.name(),
+location.getServerInstance(), tls.extent, 
goal.howUnload());
+client.unloadTablet(manager.managerLock, tls.extent, 
goal.howUnload(),
+manager.getSteadyTime());
+unloaded++;
+totalUnloaded++;
+  } catch (TException tException) {
+Manager.log.warn("[{}] Failed to request tablet unload {} 
{} {}", store.name(),
+location.getServerInstance(), tls.extent, 
goal.howUnload(), tException);
+  }
 } else {
   Manager.log.warn("Could not connect to server {}", location);
 }
@@ -1036,13 +1041,19 @@ abstract class TabletGroupWatcher extends 
AccumuloDaemonThread {
 }
 tLists.assignments.addAll(tLists.assigned);
 for (Assignment a : tLists.assignments) {
-  TServerConnection client = manager.tserverSet.getConnection(a.server);
-  if (client != null) {
-client.assignTablet(manager.managerLock, a.tablet);
-  } else {
-Manager.log.warn("Could not connect to server {}", a.server);
+  try {
+TServerConnection client = manager.tserverSet.getConnection(a.server);
+if (client != null) {
+  client.assignTablet(manager.managerLock, a.tablet);
+  manager.assignedTablet(a.tablet);
+} else {
+  Manager.log.warn("Could not connect to server {} for assignment of 
{}", a.server,
+  a.tablet);
+}
+  } catch (TException tException) {
+Manager.log.warn("Could not connect to server {} for assignment of 
{}", a.server, a.tablet,
+tException);
   }
-  manager.assignedTablet(a.tablet);
 }
   }
 



(accumulo-website) branch main updated: Fix typo (duplicate path element) in layout

2024-05-01 Thread ctubbsii
This is an automated email from the ASF dual-hosted git repository.

ctubbsii 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 bbbf8dbf Fix typo (duplicate path element) in layout
bbbf8dbf is described below

commit bbbf8dbf3faca5470fad29ce86e99329cc8ffde1
Author: Christopher Tubbs 
AuthorDate: Wed May 1 18:03:09 2024 -0400

Fix typo (duplicate path element) in layout
---
 _layouts/default.html | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/_layouts/default.html b/_layouts/default.html
index 1912f40c..4b3b1f88 100644
--- a/_layouts/default.html
+++ b/_layouts/default.html
@@ -22,7 +22,7 @@
 
 
 
-
+
 
 
 



(accumulo-website) branch main updated: Update site to comply with ASF privacy site checks (#423)

2024-05-01 Thread ctubbsii
This is an automated email from the ASF dual-hosted git repository.

ctubbsii 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 b58ecebb Update site to comply with ASF privacy site checks (#423)
b58ecebb is described below

commit b58ecebb046146fb1383ffd8c15135164dc919a6
Author: Christopher Tubbs 
AuthorDate: Wed May 1 17:38:50 2024 -0400

Update site to comply with ASF privacy site checks (#423)

See https://whimsy.apache.org/site/project/accumulo

* Add link to ASF privacy policy in ASF links section of navigation
* Stop linking to external CDNs for our default page layout. Instead,
  copy the resources, so they are hosted by our site. We don't really
  need the integrity checks to verify the downloads from CDN sites
  anymore. It adds no security since we're hosting them on the same
  service that would provide the checksum (same security domain for
  users to trust), and removing them alleviates the browser from that
  unnecessary work.
* Standardize how resources are specified, so they are more consistent
  and easier to update, including converting the embedded ``
  from `_includes/scripts.html`, which contained our own custom scripts
  for our site, into its own `.js` file.
* Include free web fonts from fontawesome project, so the css can link
  to the ones we're hosting (downloaded from
  https://fontawesome.com/download, specifically
  
https://use.fontawesome.com/releases/v6.4.2/fontawesome-free-6.4.2-web.zip)

This fixes #420
---
 _includes/nav.html |   1 +
 _includes/scripts.html |  28 
 _layouts/default.html  |  17 ++-
 css/bootstrap/5.3.1/dist/css/bootstrap.min.css |   6 +
 css/datatables/bs5/dt-1.13.6/datatables.min.css|  19 +++
 .../fontawesome-free-6.4.2-web/LICENSE.txt | 165 +
 .../fontawesome-free-6.4.2-web/css/all.min.css |   9 ++
 .../webfonts/fa-brands-400.ttf | Bin 0 -> 189684 bytes
 .../webfonts/fa-brands-400.woff2   | Bin 0 -> 109808 bytes
 .../webfonts/fa-regular-400.ttf| Bin 0 -> 63348 bytes
 .../webfonts/fa-regular-400.woff2  | Bin 0 -> 24488 bytes
 .../webfonts/fa-solid-900.ttf  | Bin 0 -> 394668 bytes
 .../webfonts/fa-solid-900.woff2| Bin 0 -> 150020 bytes
 .../webfonts/fa-v4compatibility.ttf| Bin 0 -> 10172 bytes
 .../webfonts/fa-v4compatibility.woff2  | Bin 0 -> 4568 bytes
 js/accumulo.js |  29 
 js/bootstrap/5.3.1/dist/js/bootstrap.bundle.min.js |   7 +
 js/datatables/bs5/dt-1.13.6/datatables.min.js  |  22 +++
 js/jquery/3.7.0/jquery.min.js  |   2 +
 19 files changed, 269 insertions(+), 36 deletions(-)

diff --git a/_includes/nav.html b/_includes/nav.html
index 015d265a..3c10fc71 100644
--- a/_includes/nav.html
+++ b/_includes/nav.html
@@ -56,6 +56,7 @@
 
  • https://www.apache.org/security";>Security
  • https://www.apache.org/foundation/thanks";>Thanks
  • https://www.apache.org/foundation/policies/conduct";>Code of Conduct
  • +
  • https://www.apache.org/foundation/policies/privacy.html";>Privacy Policy
  • https://www.apache.org/events/current-event.html";>Current Event
  • diff --git a/_includes/scripts.html b/_includes/scripts.html deleted file mode 100644 index 8a3db891.. --- a/_includes/scripts.html +++ /dev/null @@ -1,28 +0,0 @@ -