Re: Review Request 29898: HIVE-9298: Support reading alternate timestamp formats

2015-01-31 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29898/#review70512 --- Ship it! Ship It! - Ashutosh Chauhan On Jan. 31, 2015, 2:25

Re: Review Request 29898: HIVE-9298: Support reading alternate timestamp formats

2015-01-30 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29898/ --- (Updated Jan. 31, 2015, 2:25 a.m.) Review request for hive and Ashutosh

Re: Review Request 29898: HIVE-9298: Support reading alternate timestamp formats

2015-01-30 Thread Jason Dere
On Jan. 28, 2015, 1:22 a.m., Ashutosh Chauhan wrote: serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazySimpleSerDe.java, line 649 https://reviews.apache.org/r/29898/diff/2/?file=825983#file825983line649 I think there is a helper method in apache commons (or guava) which can

Re: Review Request 29898: HIVE-9298: Support reading alternate timestamp formats

2015-01-28 Thread Jason Dere
On Jan. 28, 2015, 1:22 a.m., Ashutosh Chauhan wrote: common/pom.xml, lines 59-63 https://reviews.apache.org/r/29898/diff/2/?file=825966#file825966line59 Since joda jar will be shipped to task nodes, this needs to be added in hive-exec jar. I think we keep that list in one of the

Re: Review Request 29898: HIVE-9298: Support reading alternate timestamp formats

2015-01-28 Thread Ashutosh Chauhan
On Jan. 28, 2015, 1:22 a.m., Ashutosh Chauhan wrote: common/pom.xml, lines 59-63 https://reviews.apache.org/r/29898/diff/2/?file=825966#file825966line59 Since joda jar will be shipped to task nodes, this needs to be added in hive-exec jar. I think we keep that list in one of the

Re: Review Request 29898: HIVE-9298: Support reading alternate timestamp formats

2015-01-27 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29898/#review69930 --- common/pom.xml https://reviews.apache.org/r/29898/#comment114765

Re: Review Request 29898: HIVE-9298: Support reading alternate timestamp formats

2015-01-19 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29898/ --- (Updated Jan. 20, 2015, 12:34 a.m.) Review request for hive and Ashutosh

Review Request 29898: HIVE-9298: Support reading alternate timestamp formats

2015-01-14 Thread Jason Dere
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/29898/ --- Review request for hive and Ashutosh Chauhan. Bugs: HIVE-9298