Log In - Home Page

Patch: Add more regression tests for SCHEMA

Edit Patch - Move To Another CommitFest - Delete Patch

CommitFest 2013-06
Topic Regression Tests
Patch Status Returned with Feedback
Author Robins Tharakan
Reviewers Fabien Coelho
Committer Nobody
Close Date 2013-07-16
Patch by robins on 2013-03-18 10:42:44 PM: Initial version.
Review by calvin on 2013-05-03 01:39:14 PM: Looked at and tested, ok. Suggestions: always use different schema names in different tests; when checking owner, use the schema name instead of filtering others?
Patch by robins on 2013-05-07 11:28:00 PM: Patch updated to use different schema / role names for different tests
Review by calvin on 2013-05-08 07:39:48 AM: Ok + 2 questions
Patch by robins on 2013-06-17 11:03:12 PM: Updated the patch taking the two comments above into consideration.
Review by calvin on 2013-06-26 02:49:07 AM: Ok, v4 addresses my remarks.
Review by rhaas on 2013-07-03 03:19:37 PM: sundry issues
Patch by robins on 2013-07-07 01:20:50 PM: Updated as per feedback
Comment by calvin on 2013-07-09 04:02:18 PM: Patch format issue?
Patch by robins on 2013-07-15 01:13:14 PM: Attached updated patch. (The last line(-ending?) seems to have been missing).

Add Comment

Please log in to comment on this patch.