https://bugzilla.wikimedia.org/show_bug.cgi?id=48875

Brad Jorsch <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #7 from Brad Jorsch <[email protected]> ---
I did a little digging into this issue today.

<http://bugs.mysql.com/bug.php?id=7014> (the one mentioning "SAP") makes it
sound like the behavior we *want*: SHOW VIEW on the view *or* SELECT on the
underlying tables will allow EXPLAIN.

<http://bugs.mysql.com/bug.php?id=64198> isn't really related. The complaint
there is that having SELECT on everything is no longer enough, SHOW VIEW is now
required too. This is the opposite of what we want, although the same
underlying change probably caused it.

So I tracked down the commit that it looks like actually made it be required to
have SELECT on the underlying tables:
<https://bazaar.launchpad.net/~maria-captains/maria/10.0/revision/1810.4002.32>.
You can see the change in behavior in the unit test file
mysql-test/t/view_grant.test: look at added line 206, before having SELECT and
SHOW VIEW on the view and nothing on the underlying table was wanting a
successful query and there it's being changed to expect the error.

It references a private (ugh) bug, but from the commit summary and comments
added in the patch it seems that the "problem" being fixed there is that
allowing EXPLAIN with only SHOW VIEW on the view was allowing people to get an
estimate of how many rows were in the underlying table. Oh noes!

That patch also points us right at the bit of code that would need to be
changed if someone wants to try getting the MariaDB people to change this.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
Wikibugs-l mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/wikibugs-l

Reply via email to