Hi!
I'm pretty much like the idea of the patch. Looks like an overlook in SQL standard for me.
Anyway, patch apply with no conflicts and implements described functionality.
I don't like this part of the patch at all. Not only is the
documentation only half baked, but the entire concept of the two
commands is different. Especially since I believe the command should
also create a generated column from a non-generated one.
But I have to agree with Vik Fearing, we can make this patch better, should we?
I totally understand your intentions to keep the code flow simple and reuse existing code as much
as possible. But in terms of semantics of these commands, they are quite different from each other.
And in terms of reading of the code, this makes it even harder to understand what is going on here.
So, in my view, consider split these commands.
Hope, that helps. Again, I'm +1 for this patch.
-- Best regards,
Maxim Orlov.
