Github user traflm commented on a diff in the pull request:
https://github.com/apache/trafodion/pull/1485#discussion_r177260101
--- Diff: core/sql/common/ComResWords.cpp ---
@@ -395,8 +395,6 @@ const ComResWord ComResWords::resWords_[] = {
ComResWord("USAGE", ANS_|RESWORD_),
ComResWord("USER", ANS_|RESWORD_),
ComResWord("USING", ANS_|RESWORD_),
- ComResWord("UUID", ANS_|RESWORD_),
- ComResWord("VALUE", ANS_|RESWORD_),
--- End diff --
Yes, @DaveBirdsall , Let me revert the changes to 'VALUE'.
The reason is we found 'VALUE' is also a popular string for column/table
names and there is no strict reason to keep it as reserved word.
And we changed it in a commercial branch and test and no side effect. So I
want to change it together with 'UUID' ( UUID for sure is very popular in the
naming of table/columns )
But for clarity, let me change it, unless you change your mind and allow
this :-)
---
