[GitHub] incubator-quickstep pull request #108: Remove unnecessary code from FastSepa...

2016-10-06 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/incubator-quickstep/pull/108 --- 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 enabled and wishes so, or if the

[GitHub] incubator-quickstep issue #108: Remove unnecessary code from FastSeparateCha...

2016-10-06 Thread pateljm
Github user pateljm commented on the issue: https://github.com/apache/incubator-quickstep/pull/108 LGTM. Merging since Travis was run before. --- 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

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

2016-10-06 Thread pateljm
Github user pateljm commented on the issue: https://github.com/apache/incubator-quickstep/pull/103 @zuyu I'd suggest move this feature along and not expand the scope unnecessarily. There is a tradeoff -- one can dump the line numbers of the problematic tuples, but that slows down