HoustonPutman commented on code in PR #2062:
URL: https://github.com/apache/solr/pull/2062#discussion_r1911647147
##########
gradle/testing/randomization.gradle:
##########
@@ -199,7 +199,12 @@ allprojects {
// Enable security manager, if requested. We could move the selection
of security manager and security policy
// to each project's build/ configuration but it seems compact enough
to keep it here for now.
- if
(Boolean.parseBoolean(testOptionsResolved["tests.useSecurityManager"])) {
+ def useSecurityManager =
Boolean.parseBoolean(testOptionsResolved["tests.useSecurityManager"]);
Review Comment:
Are you talking about the `testOptionsResolved` thing? If so it's all over
this randomization grade file. I have no idea why it's used but I just added
the ability for a project to override whatever it is.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org
