[GitHub] carbondata pull request #1349: [CARBONDATA-1470] csv data should not show in...

classic Classic list List threaded Threaded
5 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #1349: [CARBONDATA-1470] csv data should not show in...

qiuchenjian-2
GitHub user kapilreja opened a pull request:

    https://github.com/apache/carbondata/pull/1349

    [CARBONDATA-1470] csv data should not show in error log when data column length is greater than 100000 characters

    Added method to handle TextParsingException and stop the leak of the sensitive data in logs

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

    $ git pull https://github.com/kapilreja/carbondata DTS_01

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

    https://github.com/apache/carbondata/pull/1349.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 #1349
   
----
commit d384a0df6ed45d156512e7db686532527f2fe3c2
Author: kapilreja <[hidden email]>
Date:   2017-09-12T06:32:36Z

    Added method to handle TextParsingException and stop the leak of the sensitive data in logs.

----


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #1349: [CARBONDATA-1470] csv data should not show in error ...

qiuchenjian-2
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1349
 
    Can one of the admins verify this patch?


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #1349: [CARBONDATA-1470] csv data should not show in error ...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user CarbonDataQA commented on the issue:

    https://github.com/apache/carbondata/pull/1349
 
    Can one of the admins verify this patch?


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata issue #1349: [CARBONDATA-1470] csv data should not show in error ...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/1349
 
    LGTM


---
Reply | Threaded
Open this post in threaded view
|

[GitHub] carbondata pull request #1349: [CARBONDATA-1470] csv data should not show in...

qiuchenjian-2
In reply to this post by qiuchenjian-2
Github user asfgit closed the pull request at:

    https://github.com/apache/carbondata/pull/1349


---