[GitHub] carbondata pull request #2368: [CARBONDATA-2603] Fix: error handling during ...

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

[GitHub] carbondata pull request #2368: [CARBONDATA-2603] Fix: error handling during ...

qiuchenjian-2
GitHub user ajantha-bhat opened a pull request:

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

    [CARBONDATA-2603] Fix: error handling during reader build failure

    problem : When the CarbonReaderBuilder.build() is failed due to some problems like invalid projection that leads to query model creation failure. Blocklet datamap is not cleared for that table.So, the next reader instance uses old blocklet datamap . That creates error.
   
    Solution: Clear the blocklet datamap if the reader build is failed.
   
   
    Be sure to do all of the following checklist to help us incorporate
    your contribution quickly and easily:
   
     - [ ] Any interfaces changed? NA
     
     - [ ] Any backward compatibility impacted? NA
     
     - [ ] Document update required? NA
   
     - [ ] Testing done
    updated the UT test cases      
     - [ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA. NA
   


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

    $ git pull https://github.com/ajantha-bhat/carbondata issue_fix

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

    https://github.com/apache/carbondata/pull/2368.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 #2368
   
----
commit 1fbdcdde84fc8a7650d63945a950e5fb59f665cf
Author: ajantha-bhat <ajanthabhat@...>
Date:   2018-06-11T13:47:33Z

    [CARBONDATA-2603] Fix: error handling during reader build failure

----


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

[GitHub] carbondata issue #2368: [CARBONDATA-2603] Fix: error handling during reader ...

qiuchenjian-2
Github user ravipesala commented on the issue:

    https://github.com/apache/carbondata/pull/2368
 
    SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/5264/



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

[GitHub] carbondata issue #2368: [CARBONDATA-2603] Fix: error handling during reader ...

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

    https://github.com/apache/carbondata/pull/2368
 
    Build Failed  with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6305/



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

[GitHub] carbondata issue #2368: [CARBONDATA-2603] Fix: error handling during reader ...

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

    https://github.com/apache/carbondata/pull/2368
 
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5143/



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

[GitHub] carbondata issue #2368: [CARBONDATA-2603] Fix: error handling during reader ...

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

    https://github.com/apache/carbondata/pull/2368
 
    retest this please


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

[GitHub] carbondata issue #2368: [CARBONDATA-2603] Fix: error handling during reader ...

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

    https://github.com/apache/carbondata/pull/2368
 
    Build Success with Spark 2.2.1, Please check CI http://88.99.58.216:8080/job/ApacheCarbonPRBuilder/5146/



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

[GitHub] carbondata issue #2368: [CARBONDATA-2603] Fix: error handling during reader ...

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

    https://github.com/apache/carbondata/pull/2368
 
    Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/6308/



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

[GitHub] carbondata issue #2368: [CARBONDATA-2603] Fix: error handling during reader ...

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

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


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

[GitHub] carbondata pull request #2368: [CARBONDATA-2603] Fix: error handling during ...

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

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


---