You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@asterixdb.apache.org by im...@apache.org on 2026/08/11 23:02:18 UTC
(asterixdb) branch master updated (f5d90d5649 -> f9fc39c211)
This is an automated email from the ASF dual-hosted git repository.
parshimers pushed a change to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git
from f5d90d5649 [ASTERIXDB-3771][COMP] Fix ANN top-k projection collision with meta()
new 812a051316 [NO ISSUE][CONF] Remove redundant --add-opens args
new f9fc39c211 [NO ISSUE][UI] Always use Bun for Dashboard Build
The 2 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:
asterixdb/asterix-dashboard/src/node/bunfig.toml | 3 +++
.../org/apache/hyracks/control/nc/service/NCService.java | 14 --------------
2 files changed, 3 insertions(+), 14 deletions(-)
(asterixdb) 02/02: [NO ISSUE][UI] Always use Bun for Dashboard Build
Posted by im...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
parshimers pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git
commit f9fc39c2119f50a56be2cec48f399a3e5a15791b
Author: Ian Maxon <im...@apache.org>
AuthorDate: Tue Aug 11 12:28:47 2026 -0700
[NO ISSUE][UI] Always use Bun for Dashboard Build
Details:
If node is present on a machine being used to build
the dashboard, the Angular build (ng) can potentially
use it instead of the installed version of Bun, even
though we use Bun to run the script to begin with.
Forcing use of Bun through bunfig.toml fixes this
without requiring other environmental changes.
Change-Id: I719673088db0fc809ec9ed8b208ae418a90b3d91
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21538
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Ian Maxon <im...@apache.org>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Suryaa Charan Shivakumar <ss...@ucr.edu>
---
asterixdb/asterix-dashboard/src/node/bunfig.toml | 3 +++
1 file changed, 3 insertions(+)
diff --git a/asterixdb/asterix-dashboard/src/node/bunfig.toml b/asterixdb/asterix-dashboard/src/node/bunfig.toml
index 732af8eae8..b312d52b40 100644
--- a/asterixdb/asterix-dashboard/src/node/bunfig.toml
+++ b/asterixdb/asterix-dashboard/src/node/bunfig.toml
@@ -16,5 +16,8 @@
# specific language governing permissions and limitations
# under the License.
# ------------------------------------------------------------
+[run]
+bun = true
+
[install]
minimumReleaseAge = 604800 #7 days in seconds, to hopefully not be pwned
(asterixdb) 01/02: [NO ISSUE][CONF] Remove redundant --add-opens args
Posted by im...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
parshimers pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/asterixdb.git
commit 812a051316422fd53ec5fdbde7c1426151578d18
Author: Ian Maxon <im...@apache.org>
AuthorDate: Fri Jul 31 12:09:35 2026 -0700
[NO ISSUE][CONF] Remove redundant --add-opens args
- user model changes: no
- storage format changes: no
- interface changes: no
Details:
Now that the required --add-opens arguments are being
put into the appassembler scripts, having the NCService
add them when starting an NC is redundant at best.
Change-Id: I13ec2c1063754b93d1fad551362134c0d1c593d6
Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21478
Reviewed-by: Ian Maxon <im...@apache.org>
Tested-by: Jenkins <je...@fulliautomatix.ics.uci.edu>
Reviewed-by: Michael Blow <mb...@apache.org>
Integration-Tests: Jenkins <je...@fulliautomatix.ics.uci.edu>
---
.../org/apache/hyracks/control/nc/service/NCService.java | 14 --------------
1 file changed, 14 deletions(-)
diff --git a/hyracks-fullstack/hyracks/hyracks-control/hyracks-nc-service/src/main/java/org/apache/hyracks/control/nc/service/NCService.java b/hyracks-fullstack/hyracks/hyracks-control/hyracks-nc-service/src/main/java/org/apache/hyracks/control/nc/service/NCService.java
index 96d3ecbcd8..7fe2058dcb 100644
--- a/hyracks-fullstack/hyracks/hyracks-control/hyracks-nc-service/src/main/java/org/apache/hyracks/control/nc/service/NCService.java
+++ b/hyracks-fullstack/hyracks/hyracks-control/hyracks-nc-service/src/main/java/org/apache/hyracks/control/nc/service/NCService.java
@@ -31,7 +31,6 @@ import java.net.InetAddress;
import java.net.ServerSocket;
import java.net.Socket;
import java.util.ArrayList;
-import java.util.Arrays;
import java.util.Date;
import java.util.List;
import java.util.Map;
@@ -86,12 +85,6 @@ public class NCService {
*/
private static OperatingSystemMXBean osMXBean;
- public static final String[] DEFAULT_ADD_OPENS =
- { "--add-opens=jdk.management/com.sun.management.internal=ALL-UNNAMED",
- "--add-opens=java.management/sun.management=ALL-UNNAMED",
- "--add-opens=java.base/java.lang=ALL-UNNAMED", "--add-opens=java.base/java.nio=ALL-UNNAMED",
- "--add-opens=java.base/java.util.concurrent.locks=ALL-UNNAMED" };
-
private static List<String> buildCommand() throws IOException {
List<String> cList = new ArrayList<>();
@@ -148,13 +141,6 @@ public class NCService {
jvmargs = jvmargs + " -Xmx" + heapSize + "m";
}
- // Squelch some module access warnings and errors from JDK9+
- if (!jvmargs.contains("-add-opens")) {
- StringBuilder jvmArgsBuilder = new StringBuilder(jvmargs);
- Arrays.stream(DEFAULT_ADD_OPENS).map(s -> jvmArgsBuilder.append(s));
- jvmargs = jvmArgsBuilder.toString();
- }
-
env.put("JAVA_OPTS", jvmargs.trim());
LOGGER.info("Setting JAVA_OPTS to " + jvmargs);
}