ignite-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [ignite] dgarus commented on a change in pull request #6707: IGNITE-11410
Date Mon, 14 Oct 2019 11:06:17 GMT
dgarus commented on a change in pull request #6707: IGNITE-11410
URL: https://github.com/apache/ignite/pull/6707#discussion_r334428700
 
 

 ##########
 File path: modules/core/src/main/java/org/apache/ignite/internal/processors/cache/CacheOperationContext.java
 ##########
 @@ -33,8 +34,8 @@
 public class CacheOperationContext implements Serializable {
     /** */
     //TODO IGNITE-8801 remove this and set default as `false`.
-    public static final boolean DFLT_ALLOW_ATOMIC_OPS_IN_TX =
-        IgniteSystemProperties.getBoolean(IGNITE_ALLOW_ATOMIC_OPS_IN_TX, true);
+    public static final boolean DFLT_ALLOW_ATOMIC_OPS_IN_TX = SecurityUtils.doPrivileged(
 
 Review comment:
   We don't put property reading to the sandbox in that place.
   We allow reading the property with permissions that granted to ProtecionDoman of CacheOperationContext
class.
   This change is important and related to the feature, IgniteOperationsInsideSandboxTest
shows it.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

Mime
View raw message