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 2018/03/13 20:04:47 UTC

[GitHub] trafodion pull request #1473: [TRAFODION-2977] Fix issues with maximum key l...

GitHub user DaveBirdsall opened a pull request:

    https://github.com/apache/trafodion/pull/1473

    [TRAFODION-2977] Fix issues with maximum key length detection

    This set of changes fixes several minor bugs:
    
    1. The key length limit that Trafodion imposes differed from the HBase limit. They are now the same.
    
    2. CREATE INDEX logic lacked a check on key length. This has now been added.
    
    3. There was a typo in the message text for message 1141 preventing the computed key length from being displayed. This has been fixed.
    
    4. There was a bug in the generator where, if CREATE INDEX were allowed to have a long key, we generated plans to populate that index that would overrun their buffers, causing ESPs to core. This has been fixed; we now guarantee that buffers are long enough for at least one row.

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

    $ git pull https://github.com/DaveBirdsall/trafodion Trafodion2977

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

    https://github.com/apache/trafodion/pull/1473.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 #1473
    
----
commit e7fd657474842b017e26f50aeda3895b7cfd337c
Author: Dave Birdsall <db...@...>
Date:   2018-03-13T20:00:18Z

    [TRAFODION-2977] Fix issues with maximum key length detection

----


---

[GitHub] trafodion pull request #1473: [TRAFODION-2977] Fix issues with maximum key l...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/trafodion/pull/1473


---