[GitHub] incubator-quickstep issue #103: QUICKSTEP-46 Fault tolerance in bulk loading...

2016-10-28 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/103 @pateljm Rebase done. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] incubator-quickstep issue #103: QUICKSTEP-46 Fault tolerance in bulk loading...

2016-10-27 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/103 Rebased it again to be in synch with the latest commits @hbdeshmukh @pateljm . Please merge this PR if everything looks okay. Thanks ! --- If your project is set up for it, you

[GitHub] incubator-quickstep issue #103: QUICKSTEP-46 Fault tolerance in bulk loading...

2016-10-20 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/103 @pateljm Rebased it. Please merge this PR if it is good to go now. Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-quickstep issue #103: QUICKSTEP-46 Fault tolerance in bulk loading...

2016-10-10 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/103 @zuyu Thanks. I have now included the test in cmake file. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] incubator-quickstep issue #103: QUICKSTEP-46 Fault tolerance in bulk loading...

2016-10-09 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/103 Thanks for demonstrating me the testing infrastructure @hakanmemisoglu and @hbdeshmukh. As discussed I have added a test case where some faulty records are present. Also logged the

[GitHub] incubator-quickstep issue #103: QUICKSTEP-46 Fault tolerance in bulk loading...

2016-10-03 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/103 @pateljm Thanks. I will add the suggested test after a discussion with @hbdeshmukh @hakanmemisoglu about the testing infrastructure. @zuyu Thanks for your suggestion. Moving the

[GitHub] incubator-quickstep pull request #103: QUICKSTEP-46 Fault tolerance in bulk ...

2016-09-29 Thread tarunbansal
GitHub user tarunbansal opened a pull request: https://github.com/apache/incubator-quickstep/pull/103 QUICKSTEP-46 Fault tolerance in bulk loading data This PR helps add fault tolerance while bulk loading data from a file. Now quickstep will skip a record(a row) if it is faulty and

[GitHub] incubator-quickstep issue #99: QUICKSTEP-36 Quickstep gives segmentation fau...

2016-09-14 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/99 Thanks @hbdeshmukh and @zuyu for the suggestions and help. I have squashed all the commits into one. Please merge this PR if it is good to go now. --- If your project is set up for

[GitHub] incubator-quickstep issue #99: QUICKSTEP-36 Quickstep gives segmentation fau...

2016-09-12 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/99 Hi @zuyu Thanks for the welcome and the information about coding style. I have made the suggested changes and reverted file permission change which was done erroneously in the last

[GitHub] incubator-quickstep issue #99: QUICKSTEP-36 Quickstep gives segmentation fau...

2016-09-08 Thread tarunbansal
Github user tarunbansal commented on the issue: https://github.com/apache/incubator-quickstep/pull/99 Thanks @hbdeshmukh for noticing this possible ambiguity. I am still digging around the code and will update the PR once the fix is ready. --- If your project is set up for it, you

[GitHub] incubator-quickstep pull request #99: QUICKSTEP-36 Quickstep gives segmentat...

2016-09-07 Thread tarunbansal
GitHub user tarunbansal opened a pull request: https://github.com/apache/incubator-quickstep/pull/99 QUICKSTEP-36 Quickstep gives segmentation fault when it imports table from a file that does not have access rights. This PR fixes QUICKSTEP-36 - Perform access check on a