Tarball download slows down and occasionally fails CI builds. Hive is now available from maven.
piggybank unit tests:
Total time: 12 minutes 48 seconds
Generally looks good.
A couple of comments:
Is the modification to HiveColumnarLoader.java intended and related to this patch?
Could you delete, rather than comment, the code that you removed in the path creation?
Updated patch. The Hive ColumnarStruct API has changed (getField no longer has a org.apache.hadoop.io.Text nullSequence parameter). I removed the member declaration as well.
Patch looks good.
Committed to trunk.
Is there a compelling reason to put the patch also in the 0.10 branch?
Note: Hive version bumped from 0.4.1 to 0.8 with this patch.
I forgot to say: feel free to reopen the issue if you feel this patch is needed in the 0.10 branch.
Thanks Gianmarco - please commit to 0.10 also, we have seen intermittent issues with the hive download that fail our 0.9 and 0.10 CI builds, so we would like to have this change in the branch.
Also commit to 0.9/0.10 as per Thomas request.