Merge lp://qastaging/~stewart/drizzle/bug710818-show-table-generator into lp://qastaging/drizzle/7.0
Proposed by
Stewart Smith
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Brian Aker | ||||
Approved revision: | 2189 | ||||
Merged at revision: | 2196 | ||||
Proposed branch: | lp://qastaging/~stewart/drizzle/bug710818-show-table-generator | ||||
Merge into: | lp://qastaging/drizzle/7.0 | ||||
Diff against target: |
364 lines (+272/-1) 12 files modified
plugin/show_dictionary/show_columns.cc (+7/-0) plugin/show_dictionary/show_create_schema.cc (+7/-0) plugin/show_dictionary/show_create_table.cc (+10/-1) plugin/show_dictionary/show_indexes.cc (+7/-0) plugin/simple_user_policy/tests/r/show_columns.result (+28/-0) plugin/simple_user_policy/tests/r/show_indexes.result (+28/-0) plugin/simple_user_policy/tests/r/show_schema.result (+24/-0) plugin/simple_user_policy/tests/r/show_table.result (+30/-0) plugin/simple_user_policy/tests/t/show_columns.test (+33/-0) plugin/simple_user_policy/tests/t/show_indexes.test (+33/-0) plugin/simple_user_policy/tests/t/show_schema.test (+30/-0) plugin/simple_user_policy/tests/t/show_table.test (+35/-0) |
||||
To merge this branch: | bzr merge lp://qastaging/~stewart/drizzle/bug710818-show-table-generator | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Drizzle Developers | Pending | ||
Review via email: mp+50843@code.qastaging.launchpad.net |
Description of the change
add permissions checks to the SHOW commands.
Each check is rather simple, and since we're not iterating over anything, we just embed the check instead of having a generator that provides a permission checked iterator.
http://
To post a comment you must log in.