----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16403/#review39269 -----------------------------------------------------------
ql/src/test/org/apache/hadoop/hive/ql/WindowsPathUtil.java <https://reviews.apache.org/r/16403/#comment71629> For what it's worth, it seems the if (windows) should stay with the caller of the method. That is, caller should do: if( windows) { WindowsPathUtil.convertPaths(); } Same for getHdfsUriString(). - Xuefu Zhang 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 > >
