EXPEbdodla / hive-json-serde

Automatically exported from code.google.com/p/hive-json-serde
0 stars 0 forks source link

Version hive-json-serde.jar #6

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Checkout the project
2. Build with "ant build"
3. Observe the created artifact is build/hive-json-serde.jar

What is the expected output? What do you see instead?
The expected output would have a version number, so it can be referenced with 
ivy.

What version of the product are you using? On what operating system?
Unknown version :)

Please provide any additional information below.
One of my coworkers made a few changes to this project at:

    https://github.com/johanoskarsson/hive-json-serde

I pinged him about adding versioning, and got pointed at this project since 
we'd like to get all changes merged back here. We have this working 
experimentally but would like to use in production, and need to push the jar 
into ivy so we can build, which is when versioning came up. I see there are 
some versioned jar's published in the downloads section here, but we'd like to 
build from source. Any objection to building versioned jar's?

Original issue reported on code.google.com by traviscr...@gmail.com on 6 Jan 2011 at 7:05

GoogleCodeExporter commented 8 years ago
Adding versioning is possible, but it means that in Hive, the command:
  ADD JAR /path-to/hive-json-serde.jar;

would change based on the version, so I opted to always have the file name be 
the same. The only reason I version the files here is so that people to just 
grab the jar and not the source know an upgrade is needed.

Do you know of a good way to have the best of both worlds in ant? I guess we 
could just make a copy after the build succeeded.

Original comment by pas...@gmail.com on 31 Jul 2011 at 5:02

GoogleCodeExporter commented 8 years ago
There are now 2 JAR files created - one versioned, and one not. Committed with 
r29.

Original comment by pas...@gmail.com on 31 Jul 2011 at 5:55

GoogleCodeExporter commented 8 years ago
Oh, and with r32, I have merged the changes johanoskarsson made to handle 
arrays (but not to handle 'dodgy data' as I think that is specific to your data 
set.

Original comment by pas...@gmail.com on 31 Jul 2011 at 6:57