Hi!

On 16.05.2016 06:00, Alexander Barkov wrote:
Hi Sanja, Sergei,


Sanja, you asked me to review a patch for MDEV-10035:

revision-id: 4ffe2295e78538dde93df078421726f0c5a7d2a2 (mariadb-10.2.0-29-g4ffe229)
parent(s): b79944950e5e5db40cf7ad49061edf5f105512c4
committer: Oleksandr Byelkin
timestamp: 2016-05-15 15:25:33 +0200
message:


I earlier also proposed the same idea to disallow FOR UPDATE,
and even created a patch disallowing this in the parser
syntactically (see attached).


But Sergei worried that we should not do it this way and proposed some other solutions. Please see Sergei's comments in:

MDEV-10063 VIEWs and subqueries with FOR UPDATE


Sergei, are you still in doubts?

Just now this parameter for VIEWs effectively (and silently) ignored. So error IMHO is better.

[skip]

_______________________________________________
Mailing list: https://launchpad.net/~maria-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~maria-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to