> On April 2, 2014, 7:22 a.m., Thejas Nair wrote: > > common/src/test/org/apache/hadoop/hive/conf/TestHiveConf.java, line 38 > > <https://reviews.apache.org/r/16403/diff/2/?file=544942#file544942line38> > > > > should we just lowercase before comparison, only in windows ? > > > >
Sure, will make that change. - Jason ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16403/#review39243 ----------------------------------------------------------- On April 2, 2014, 2:11 a.m., Jason Dere wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16403/ > ----------------------------------------------------------- > > (Updated April 2, 2014, 2:11 a.m.) > > > Review request for hive and Thejas Nair. > > > Bugs: HIVE-5176 > https://issues.apache.org/jira/browse/HIVE-5176 > > > Repository: hive-git > > > Description > ------- > > We need to make certain changes across the board to allow us to read/parse > windows paths. Some are escaping changes, some are being strict about how we > read paths (through URL.encode/decode, etc) > > > Diffs > ----- > > common/src/test/org/apache/hadoop/hive/conf/TestHiveConf.java a31238b > ql/src/test/org/apache/hadoop/hive/ql/WindowsPathUtil.java PRE-CREATION > ql/src/test/org/apache/hadoop/hive/ql/exec/TestExecDriver.java 5991aae > > ql/src/test/org/apache/hadoop/hive/ql/metadata/TestHiveMetaStoreChecker.java > e453f56 > > Diff: https://reviews.apache.org/r/16403/diff/ > > > Testing > ------- > > > Thanks, > > Jason Dere > >
