[
https://issues.apache.org/jira/browse/DERBY-3137?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12540294
]
Rick Hillegas commented on DERBY-3137:
--------------------------------------
Hi Dag,
Thanks for the extensive patch description and regression tests. Also, thanks
for adding the SHOW ROLES command. I think it's very useful.
1) ij.jj: Comment on showRoles() refers to SYSTABLES. I think SYSROLES is
intended.
2) SetRoleNode: By the time you get to this node, NONE is understood to mean
the same thing as NULL Role. I wonder if this is the solution to your question
about disambiguating NONE and "NONE". Maybe NONE should be what's meant when
someone sets the ? to NULL in a dynamic "SET ROLE ?" statement.
3) SetRoleConstantAction: Ditto that. You could then eliminate the paragraph
which translates "NONE" to NULL.
> SQL roles: add catalog support
> ------------------------------
>
> Key: DERBY-3137
> URL: https://issues.apache.org/jira/browse/DERBY-3137
> Project: Derby
> Issue Type: New Feature
> Components: SQL
> Reporter: Dag H. Wanvik
> Assignee: Dag H. Wanvik
> Attachments: DERBY-3137.diff, DERBY-3137.diff, DERBY-3137.stat,
> DERBY-3137.txt
>
>
> As a next step after adding support for the roles syntax, I intend to
> make a patch which implements catalog support for roles,
> cf. SYS.SYSROLES described in the specification (attached to
> DERBY-2207). Also the patch should tie this support up to the parser
> support, so the role statements can be executed. Any privileges
> granted to roles would still have no effect at run-time.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
