This is an automated email from the ASF dual-hosted git repository.
domgarguilo 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 8829a225ee Correct units of duration of running compactions in monitor
(#5007)
8829a225ee is described below
commit 8829a225ee8e66336a6fc4102f111504a5b692b9
Author: Dom G. <[email protected]>
AuthorDate: Thu Oct 24 11:50:15 2024 -0400
Correct units of duration of running compactions in monitor (#5007)
* Correct units of duration of running compactions in monitor
---
.../src/main/resources/org/apache/accumulo/monitor/resources/js/ec.js | 1 +
.../resources/org/apache/accumulo/monitor/resources/js/functions.js | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git
a/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/ec.js
b/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/ec.js
index 545c2559df..21f63e2a46 100644
---
a/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/ec.js
+++
b/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/ec.js
@@ -80,6 +80,7 @@ $(document).ready(function () {
"columnDefs": [{
"targets": "duration",
"render": function (data, type, row) {
+ data = data / 1_000_000; // convert from nanos to millis
if (type === 'display') data = timeDuration(data);
return data;
}
diff --git
a/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/functions.js
b/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/functions.js
index af0363a67d..36a46f69e8 100644
---
a/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/functions.js
+++
b/server/monitor/src/main/resources/org/apache/accumulo/monitor/resources/js/functions.js
@@ -157,7 +157,7 @@ function levelFormat(level) {
/**
* Converts the time to short number and adds unit
*
- * @param {number} time Time in microseconds
+ * @param {number} time Time in milliseconds
* @return {string} The time with units
*/
function timeDuration(time) {