trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From CoderSong2015 <>
Subject [GitHub] trafodion pull request #1387: [TRAFODION-2900] Fix bug could insert more tha...
Date Wed, 10 Jan 2018 10:23:24 GMT
GitHub user CoderSong2015 opened a pull request:

    [TRAFODION-2900] Fix bug could insert more than the maximum number of…

    When insert 40 Chinese char to char(30) , of which the character is utf8 , it gets success
while it should return fail.
    SQL return the max length of the column which is calculated by bytes . The returned length
of 30 chars of utf8 character is 120 bytes. However the length of Chinese chars in utf8 maybe
is 3 bytes or 4 bytes. So 40 Chinese chars could be inserted into the database. But the char(30)
represent that 30 chars regardless of how many bytes a char costs. So I modify the T4 to judge
char numbers of the string instead of bytes of the string.
    @kevinxu021 @Weixin-Xu @selvaganesang please review the code.

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

    $ git pull master

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

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

    This closes #1387
commit e4b0203c41aecf111198431e31c5106e384f4143
Author: <403438485@...>
Date:   2018-01-10T18:02:37Z

    [TRAFODION-2900] Fix bug could insert more than the maximum number of characters with



View raw message