Enhance JVM argument handling: add default heap size and native access flag

Author: tae898

bindings/python/src/arcadedb_embedded/jvm.py

@@ -151,11 +151,20 @@ def _build_jvm_args() -> list[str]:
         # 2. Headless mode if not set (Critical for server environments)
         if not any(arg.startswith("-Djava.awt.headless=") for arg in jvm_args):
             jvm_args.append("-Djava.awt.headless=true")
+
+        # 3. Default heap if user did not set one
+        if not any(arg.startswith("-Xmx") for arg in jvm_args):
+            jvm_args.append("-Xmx4g")
+
+        # 4. Allow native access for JPype (required by newer JDKs)
+        if not any(arg.startswith("--enable-native-access") for arg in jvm_args):
+            jvm_args.append("--enable-native-access=ALL-UNNAMED")
     else:
         # Default: 4GB heap, headless mode, SIMD vector support
         jvm_args = [
             "-Xmx4g",
             "-Djava.awt.headless=true",
+            "--enable-native-access=ALL-UNNAMED",
             "--add-modules=jdk.incubator.vector",
         ]
 

bindings/python/tests/test_jvm_args.py

@@ -11,6 +11,7 @@ def test_defaults_no_env_vars():
         assert "-Xmx4g" in args
         assert "-Djava.awt.headless=true" in args
         assert "--add-modules=jdk.incubator.vector" in args
+        assert "--enable-native-access=ALL-UNNAMED" in args
         # Should have default error log
         assert any("hs_err_pid" in arg for arg in args)
 
@@ -27,21 +28,32 @@ def test_custom_jvm_args_merging():
         # Mandatory args injected
         assert "-Djava.awt.headless=true" in args
         assert "--add-modules=jdk.incubator.vector" in args
+        assert "--enable-native-access=ALL-UNNAMED" in args
+
+
+def test_custom_jvm_args_injects_heap_default_when_missing():
+    """Ensure we add a heap default if user omits -Xmx."""
+    with patch.dict(os.environ, {"ARCADEDB_JVM_ARGS": "-Dfoo=bar"}, clear=True):
+        args = _build_jvm_args()
+        assert "-Xmx4g" in args
+        assert "-Dfoo=bar" in args
 
 
 def test_custom_jvm_args_no_duplicates():
     """Test that we don't duplicate flags if user provides them."""
-    custom_args = "-Xmx2g -Djava.awt.headless=false --add-modules=jdk.incubator.vector"
+    custom_args = "-Xmx2g -Djava.awt.headless=false --add-modules=jdk.incubator.vector --enable-native-access=ALL-UNNAMED"
     with patch.dict(os.environ, {"ARCADEDB_JVM_ARGS": custom_args}, clear=True):
         args = _build_jvm_args()
 
         # Should NOT add defaults if present
         # Count occurrences
         modules_count = sum(1 for a in args if "jdk.incubator.vector" in a)
         headless_count = sum(1 for a in args if "headless" in a)
+        native_count = sum(1 for a in args if "enable-native-access" in a)
 
         assert modules_count == 1
         assert headless_count == 1
+        assert native_count == 1
 
         # Verify user's explicit choice is respected (e.g., they might want headless=false for some reason)
         # Note: Our logic just checks key presence, it doesn't force overwrite if key exists with different value.