You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@geode.apache.org by Jinmei Liao <ji...@pivotal.io> on 2016/05/20 18:21:13 UTC

Review Request 47654: GEODE-17: change default regionName to * instead of null.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/47654/
-----------------------------------------------------------

Review request for geode.


Repository: geode


Description
-------

GEODE-17: change default regionName to * instead of null.

* operations with no regionName specified in its permission string will need a higher level of role than the operations with regionName specified. This makes more sense than null.


Diffs
-----

  geode-core/src/main/java/com/gemstone/gemfire/cache/operations/OperationContext.java b81016d877c865f652c554e6462b436bdb8cff5b 
  geode-core/src/main/java/com/gemstone/gemfire/management/internal/security/ResourceOperationContext.java 580b6c08b7799fc7cbfc5806ee966a577a7e8b37 
  geode-core/src/test/java/com/gemstone/gemfire/management/internal/security/MemberMBeanSecurityJUnitTest.java cabf555f961d80ba942920e44c20827241bedf28 
  geode-core/src/test/java/com/gemstone/gemfire/management/internal/security/ResourceOperationContextJUnitTest.java ec89aaa5b452049b929bc95077925eaafde9883c 

Diff: https://reviews.apache.org/r/47654/diff/


Testing
-------

precheckin running


Thanks,

Jinmei Liao