Skip to content

CNDB-18108 Add system property to disable UDFs and skip SecurityManager installation (#2452)#2486

Merged
djatnieks merged 1 commit into
main-5.0from
CNDB-18283
Jul 1, 2026
Merged

CNDB-18108 Add system property to disable UDFs and skip SecurityManager installation (#2452)#2486
djatnieks merged 1 commit into
main-5.0from
CNDB-18283

Conversation

@djatnieks

Copy link
Copy Markdown
Member

Introduces the -Dcassandra.disable_user_defined_functions system property:

  • When set to true, UDF creation and execution are blocked, and ThreadAwareSecurityManager installation is skipped
  • Takes precedence over the YAML enable_user_defined_functions setting
  • Defaults to false to maintain backward compatibility
  • SecurityManager installation is skipped because it is only used for sandboxing UDFs

…er installation (#2452)

Introduces the `-Dcassandra.disable_user_defined_functions` system
property:

* When set to `true`, UDF creation and execution are blocked, and
ThreadAwareSecurityManager installation is skipped
* Takes precedence over the YAML `enable_user_defined_functions` setting
* Defaults to `false` to maintain backward compatibility
* SecurityManager installation is skipped because it is only used for
sandboxing UDFs
@github-actions

Copy link
Copy Markdown

Checklist before you submit for review

  • This PR adheres to the Definition of Done
  • Make sure there is a PR and ticket in the CNDB project updating the Converged Cassandra version
  • Use NoSpamLogger for log lines that may appear frequently in the logs
  • Verify test results on Butler
  • Test coverage for new/modified code is > 80%
  • Proper code formatting
  • Proper title for each commit staring with the project-issue number, like CNDB-1234
  • Each commit has a meaningful description
  • Each commit is not very long and contains related changes
  • Renames, moves and reformatting are in distinct commits
  • All new files should contain the IBM copyright header instead of the Apache License one (no DataStax copyright any longer)

@plpesvc-ds

Copy link
Copy Markdown

❌ Build ds-cassandra-pr-gate/PR-2486 rejected by Butler


3 regressions found
See build details here


Found 3 new test failures

Test Explanation Runs Upstream
o.a.c.index.sai.cql.VectorCompaction100dTest.testCompactionWithEnoughRowsForPQAndDeleteARow[version=ec enableNVQ=false] () NEW 🔴 0 / 30
o.a.c.index.sai.cql.VectorKeyRestrictedOnPartitionTest.partitionRestrictedTest[dc false] (compression) REGRESSION 🔴 0 / 30
o.a.c.index.sai.cql.VectorSiftSmallTest.testRerankKZeroOrderMatchesFullPrecisionSimilarity[ec true] () NEW 🔴 0 / 30

No known test failures found

@djatnieks djatnieks merged commit 222b312 into main-5.0 Jul 1, 2026
4 of 6 checks passed
@djatnieks djatnieks deleted the CNDB-18283 branch July 1, 2026 18:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants