Log In - Home Page

Patch: Largeobject access controls

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2009-09
Topic Security
Patch Status Returned with Feedback
Author KaiGai Kohei
Reviewers Jaime Casanova
Committer Nobody
Close Date 2009-10-14
Patch by kaigai on 2009-08-27 10:19:13 PM: Initial version.
Comment by kaigai on 2009-08-30 05:41:16 PM: Itagaki commented about performance and SQL gramer.
Comment by kaigai on 2009-08-30 05:43:53 PM: Tom commented data chunks should not put on toast, and default behavior of privileges.
Patch by kaigai on 2009-08-30 11:20:37 PM: Revised patch.
Comment by kaigai on 2009-09-01 02:27:14 AM: Alvaro commented that default ACL should be configurable using GUC.
KaiGai also considers the GUC parameter allows us to turn on/off access controls for largeobject to control its backward compatibility.
Patch by kaigai on 2009-09-06 07:01:44 PM: A new guc variable to turn on/off largeobject access controls
Comment by sfrost on 2009-09-18 07:40:59 PM: ping'd reviewer
Review by jcasanov on 2009-09-21 12:54:47 AM: The patch is fine as is, there are some cosmetic points (about names and such) to consider... and some ideas that maybe escape to the scope of this patch...
Patch by kaigai on 2009-09-24 06:23:28 PM: Revised patch. Some of cosmetic updates.
Comment by rhaas on 2009-09-27 11:10:41 AM: ping reviewer
Review by jcasanov on 2009-09-28 01:06:08 AM: looks good to me...
Patch by kaigai on 2009-09-28 01:58:32 AM: Revised minor stuffs based on Jaime's comments.
Patch by kaigai on 2009-10-05 08:32:18 PM: Fix patch conflicts due to the default ACL. No other changes here.
Patch by kaigai on 2009-10-07 05:54:15 PM: Revised patch, according to Tom's comment
Patch by kaigai on 2009-10-12 07:47:02 PM: Revised patch, due to the conflicts with CVS HEAD
Review by tgl on 2009-10-13 08:56:48 PM: Still needs work :-(

Add Comment

Please log in to comment on this patch.