Copilot commented on code in PR #696:
URL: https://github.com/apache/ranger/pull/696#discussion_r2670774682
##########
authz-api/src/main/java/org/apache/ranger/authz/util/RangerResourceNameParser.java:
##########
@@ -36,8 +36,9 @@
public class RangerResourceNameParser {
private static final Logger LOG =
LoggerFactory.getLogger(RangerResourceNameParser.class);
- public static final String[] EMPTY_ARRAY = new String[0];
- public static final char RRN_RESOURCE_TYPE_SEP = ':';
+ public static final String[] EMPTY_ARRAY = new String[0];
+ public static final char RRN_RESOURCE_TYPE_SEP = ':';
+ public static final char DEFAULT_RRN_RESOURCE_SEP = '/';
Review Comment:
The constant name 'DEFAULT_RRN_RESOURCE_SEP' could be clearer as
'DEFAULT_RRN_RESOURCE_SEPARATOR' to maintain consistency with other
separator-related naming in the class (e.g., 'separatorChar',
'separatorString').
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
