[GitHub] incubator-quickstep issue #139: Remove packedrow

2017-01-08 Thread hbdeshmukh
Github user hbdeshmukh commented on the issue: https://github.com/apache/incubator-quickstep/pull/139 Looks good to me. --- 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 a

[GitHub] incubator-quickstep issue #139: Remove packedrow

2017-01-06 Thread cramja
Github user cramja commented on the issue: https://github.com/apache/incubator-quickstep/pull/139 @hbdeshmukh I fixed the problem, hopefully. Waiting for Travis to come back. There's several Travis runs which I triggered (`git commit --amend; git push --force`) while te

[GitHub] incubator-quickstep issue #139: Remove packedrow

2017-01-05 Thread hbdeshmukh
Github user hbdeshmukh commented on the issue: https://github.com/apache/incubator-quickstep/pull/139 Hi @cramja - Sure, we can close this PR. Looks like Travis is failing right now and the issue is not the OOM error we saw in some of the previous PRs. Once you fix the issues, please

[GitHub] incubator-quickstep issue #139: Remove packedrow

2017-01-05 Thread cramja
Github user cramja commented on the issue: https://github.com/apache/incubator-quickstep/pull/139 @hbdeshmukh my new years resolution is to finish this PR. Want to review this with me? --- If your project is set up for it, you can reply to this email and have your reply appear on Git