----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/71016/#review216885 -----------------------------------------------------------
Thanks a lot for the patch! I had two comments, but otherwise it looks good. Nice testing btw!! :) - Marta Kuczora On July 5, 2019, 7:51 a.m., Karen Coppage wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/71016/ > ----------------------------------------------------------- > > (Updated July 5, 2019, 7:51 a.m.) > > > Review request for hive and Marta Kuczora. > > > Bugs: HIVE-21578 > https://issues.apache.org/jira/browse/HIVE-21578 > > > Repository: hive-git > > > Description > ------- > > Enable Hive to parse the following datetime formats when any combination or > subset of these or previously implemented formats is provided in one string. > > "text" (nested strings) > FM > FX > > > Diffs > ----- > > > common/src/java/org/apache/hadoop/hive/common/format/datetime/HiveSqlDateTimeFormatter.java > 998e5a2f6a > > common/src/test/org/apache/hadoop/hive/common/format/datetime/TestHiveSqlDateTimeFormatter.java > 4e822d53f9 > ql/src/test/queries/clientpositive/cast_datetime_with_sql_2016_format.q > 5a2a6d7894 > ql/src/test/results/clientpositive/cast_datetime_with_sql_2016_format.q.out > e1fd341050 > > > Diff: https://reviews.apache.org/r/71016/diff/2/ > > > Testing > ------- > > > Thanks, > > Karen Coppage > >
