trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From CoderSong2015 <...@git.apache.org>
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:

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

    [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 https://github.com/CoderSong2015/Apache-Trafodion master

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

    https://github.com/apache/trafodion/pull/1387.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 #1387
    
----
commit e4b0203c41aecf111198431e31c5106e384f4143
Author: Haolin.song <403438485@...>
Date:   2018-01-10T18:02:37Z

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

----


---

Mime
View raw message