----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62339/#review185442 -----------------------------------------------------------
geode-core/src/main/java/org/apache/geode/security/ResourcePermission.java Lines 78-91 (patched) <https://reviews.apache.org/r/62339/#comment261747> We had discussed moving towards a strictly-String API in any case. I didn't know if this should be marked `@Deprecated` or not as a result. - Patrick Rhomberg On Sept. 14, 2017, 7:05 p.m., Patrick Rhomberg wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/62339/ > ----------------------------------------------------------- > > (Updated Sept. 14, 2017, 7:05 p.m.) > > > Review request for geode. > > > Repository: geode > > > Description > ------- > > Will this preserve backwards compat? > > > Diffs > ----- > > geode-core/src/main/java/org/apache/geode/security/ResourcePermission.java > 6a920d7c217f7383e38a7465540cdc0d542b6c81 > > > Diff: https://reviews.apache.org/r/62339/diff/1/ > > > Testing > ------- > > > Thanks, > > Patrick Rhomberg > >
