maria-developers team mailing list archive

[Oleksandr Byelkin <sanja@xxxxxxxxxxxxxxxx>]

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]