avro
avro copied to clipboard
AVRO-1856 concat tool append to input
Let me know if this pull request should be pointed to a different branch.
just noticed this:
Let me know if this pull request should be pointed to a different branch.
Generally, PRs should target the master branch. In this case I'll take care of moving from 1.8 to master, presuming it doesn't explode.
👍 I'll leave this open for additional comments over the next day or two, and will then merge it in time for 1.8.2 RCs.
@busbey, do you want to merge this?
@MikeHurleySurescripts I tried to merge this on Avro master branch, but had merge conflicts, could you please apply this patch on top of the latest master branch and update the pull request accordingly? Also, Checkstyle is complaining due to trailing whitespaces, so build is broken: /Users/nkollar/work/upstream/avro/lang/java/tools/src/main/java/org/apache/avro/tool/Util.java:137: Line matches the illegal pattern 'Trailing whitespace', could you please delete the problematic whitespaces? Otherwise looks good to me!