Grokbase Groups Hive dev January 2014
FAQ
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16806/
-----------------------------------------------------------

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

Search Discussions

  • Ashutosh Chauhan at Jan 13, 2014 at 5:27 pm
    -----------------------------------------------------------
    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

Related Discussions

Discussion Navigation
viewthread | post
Discussion Overview
groupdev @
categorieshive, hadoop
postedJan 12, '14 at 1:53a
activeJan 13, '14 at 5:27p
posts2
users2
websitehive.apache.org

2 users in discussion

Xuefu Zhang: 1 post Ashutosh Chauhan: 1 post

People

Translate

site design / logo © 2022 Grokbase