Review Request: HIVE-3450 Hive maven-publish ant task should be configurable

2012-09-11 Thread Travis Crawford
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7024/ --- Review request for hive and Ashutosh Chauhan. Description --- Add Hive

Re: Review Request: HIVE-3450 Hive maven-publish ant task should be configurable

2012-09-11 Thread Travis Crawford
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7024/ --- (Updated Sept. 11, 2012, 5:21 p.m.) Review request for hive, Ashutosh Chauhan,

Re: Review Request: HIVE-3450 Hive maven-publish ant task should be configurable

2012-09-11 Thread Travis Crawford
On Sept. 11, 2012, 5:47 p.m., Feng Peng wrote: build.xml, line 23 https://reviews.apache.org/r/7024/diff/1/?file=152640#file152640line23 Any reason for this change? I changed the urn lines after reading http://maven.apache.org/ant-tasks/installation.html The following example is

Re: Review Request: HIVE-3450 Hive maven-publish ant task should be configurable

2012-09-11 Thread Travis Crawford
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7024/ --- (Updated Sept. 11, 2012, 6:30 p.m.) Review request for hive, Ashutosh Chauhan,

Re: Review Request: HIVE-3450 Hive maven-publish ant task should be configurable

2012-09-11 Thread Feng Peng
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7024/#review11355 --- lgtm. non-committer +1 - Feng Peng On Sept. 11, 2012, 6:30 p.m.,