[ http://issues.apache.org/jira/browse/DERBY-1539?page=all ]
Mamta A. Satoor updated DERBY-1539:
-----------------------------------
Attachment: DERBY1539V3statDropTriggerOnRevoke.txt
DERBY1539V3diffDropTriggerOnRevoke.txt
Dan, I have addressed your concerns in the patch
DERBY1539V3diffDropTriggerOnRevoke.txt
1)The invalidation messages are now sent by the caller rather than the method
DataDictionary.addRemovePermissionsDescriptor.
DataDictionary.addRemovePermissionsDescriptor will return true if it has made
changes to system tables such that the dependents need to be notified.
2)I have moved the drop trigger code into makeInvalid call.
3)Removed the if condition and used the existing switch.
The grant revoke tests passed with these changes.
Can someone please review/commit this patch? thanks
> As per the functional spec attached to DERBY-1330, a trigger should be
> dropped when a privilege required by the trigger is revoked.
> -----------------------------------------------------------------------------------------------------------------------------------
>
> Key: DERBY-1539
> URL: http://issues.apache.org/jira/browse/DERBY-1539
> Project: Derby
> Issue Type: New Feature
> Components: SQL
> Affects Versions: 10.2.0.0
> Reporter: Mamta A. Satoor
> Assigned To: Mamta A. Satoor
> Fix For: 10.2.0.0
>
> Attachments: DERBY1539V1hashCodeEqualsDiff.txt,
> DERBY1539V1hashCodeEqualsStat.txt, DERBY1539V2diffDropTriggerOnRevoke.txt,
> DERBY1539V2statDropTriggerOnRevoke.txt,
> DERBY1539V3diffDropTriggerOnRevoke.txt, DERBY1539V3statDropTriggerOnRevoke.txt
>
>
> A trigger tracks its privileges requirements using Derby's Dependency
> Manager. If any one of those required privileges are revoked, the trigger
> should be dropped automatically.
> I am just creating a new jira entry here so it is easier to track sub items
> of DERBY-1330. Will link this Jira entry to DERBY-1330.
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
