You are viewing a plain text version of this content. The canonical link for it is here.

Posted to codereview@trafodion.apache.org by DaveBirdsall <gi...@git.apache.org> on 2017/09/19 23:03:53 UTC

[GitHub] incubator-trafodion pull request #1239: [TRAFODION-2749] Fix issue with rese...

GitHub user DaveBirdsall opened a pull request:

    https://github.com/apache/incubator-trafodion/pull/1239

    [TRAFODION-2749] Fix issue with reserved words as column names

    The code path in UPDATE STATISTICS where it creates multi-column histograms did not handle column names that happen to be reserved words correctly. These need to be double-quoted as delimited identifiers. (Oddly, vanilla delimited identifiers worked just fine.)

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DaveBirdsall/incubator-trafodion Trafodion2749

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-trafodion/pull/1239.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #1239
    
----
commit fc575b11679862d5820584c95dc665260f42c1a9
Author: Dave Birdsall <db...@apache.org>
Date:   2017-09-19T22:58:31Z

    [TRAFODION-2749] Fix issue with reserved words as column names

----


---

[GitHub] incubator-trafodion pull request #1239: [TRAFODION-2749] Fix issue with rese...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/incubator-trafodion/pull/1239


---