----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16806/#review31619 -----------------------------------------------------------
ql/src/java/org/apache/hadoop/hive/ql/index/IndexMetadataChangeTask.java <https://reviews.apache.org/r/16806/#comment60225> This can be written as Path url = part.getDataLocation; - Ashutosh Chauhan On Jan. 12, 2014, 1:53 a.m., Xuefu Zhang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/16806/ > ----------------------------------------------------------- > > (Updated Jan. 12, 2014, 1:53 a.m.) > > > Review request for hive and Ashutosh Chauhan. > > > Bugs: HIVE-6185 > https://issues.apache.org/jira/browse/HIVE-6185 > > > Repository: hive-git > > > Description > ------- > > Fix the inconsistency and standardize it using Path. > > > Diffs > ----- > > > hcatalog/core/src/main/java/org/apache/hcatalog/cli/SemanticAnalysis/CreateTableHook.java > 791e01b > > hcatalog/core/src/main/java/org/apache/hcatalog/security/HdfsAuthorizationProvider.java > 2eba530 > > hcatalog/core/src/main/java/org/apache/hive/hcatalog/cli/SemanticAnalysis/CreateTableHook.java > 4c3acb6 > > hcatalog/core/src/test/java/org/apache/hcatalog/mapreduce/TestHCatMultiOutputFormat.java > a2be640 > > hcatalog/core/src/test/java/org/apache/hive/hcatalog/mapreduce/TestHCatMultiOutputFormat.java > 68c77c2 > > itests/util/src/main/java/org/apache/hadoop/hive/ql/hooks/VerifyOutputTableLocationSchemeIsFileHook.java > 5cc4079 > > itests/util/src/main/java/org/apache/hadoop/hive/ql/hooks/VerifyPartitionIsNotSubdirectoryOfTableHook.java > ce377be > > itests/util/src/main/java/org/apache/hadoop/hive/ql/hooks/VerifyPartitionIsSubdirectoryOfTableHook.java > 4406036 > ql/src/java/org/apache/hadoop/hive/ql/exec/ArchiveUtils.java 9608fcd > ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 9e4f1c7 > ql/src/java/org/apache/hadoop/hive/ql/exec/MoveTask.java c44e9da > ql/src/java/org/apache/hadoop/hive/ql/hooks/Entity.java f57feb9 > ql/src/java/org/apache/hadoop/hive/ql/index/IndexMetadataChangeTask.java > 364fc19 > ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 2fe86e1 > ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveMetaStoreChecker.java > 695982f > ql/src/java/org/apache/hadoop/hive/ql/metadata/Partition.java 83514a2 > ql/src/java/org/apache/hadoop/hive/ql/metadata/Table.java 0180b87 > > ql/src/java/org/apache/hadoop/hive/ql/metadata/formatting/JsonMetaDataFormatter.java > bcd75be > > ql/src/java/org/apache/hadoop/hive/ql/metadata/formatting/TextMetaDataFormatter.java > b919d1a > ql/src/java/org/apache/hadoop/hive/ql/optimizer/AbstractBucketJoinProc.java > 0991847 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/GenMRTableScan1.java > 51d56ef > ql/src/java/org/apache/hadoop/hive/ql/optimizer/IndexUtils.java 2146009 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/SamplePruner.java ab2ed81 > ql/src/java/org/apache/hadoop/hive/ql/optimizer/SimpleFetchOptimizer.java > c73c874 > > ql/src/java/org/apache/hadoop/hive/ql/optimizer/SizeBasedBigTableSelectorForAutoSMJ.java > 1a686a0 > ql/src/java/org/apache/hadoop/hive/ql/parse/DDLSemanticAnalyzer.java > 713bd54 > ql/src/java/org/apache/hadoop/hive/ql/parse/ExportSemanticAnalyzer.java > be0ad62 > ql/src/java/org/apache/hadoop/hive/ql/parse/ImportSemanticAnalyzer.java > 1ab5a60 > ql/src/java/org/apache/hadoop/hive/ql/parse/LoadSemanticAnalyzer.java > 5663fca > ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java 8e68fcf > > ql/src/java/org/apache/hadoop/hive/ql/security/authorization/StorageBasedAuthorizationProvider.java > 5557697 > ql/src/java/org/apache/hadoop/hive/ql/stats/StatsUtils.java e89e3a4 > > ql/src/test/org/apache/hadoop/hive/ql/metadata/TestHiveMetaStoreChecker.java > 69d1896 > > Diff: https://reviews.apache.org/r/16806/diff/ > > > Testing > ------- > > No new test. > > > Thanks, > > Xuefu Zhang > >
