[ 
https://issues.apache.org/jira/browse/TAJO-1493?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14900273#comment-14900273
 ] 

ASF GitHub Bot commented on TAJO-1493:
--------------------------------------

Github user jihoonson commented on a diff in the pull request:

    https://github.com/apache/tajo/pull/772#discussion_r39944352
  
    --- Diff: 
tajo-catalog/tajo-catalog-server/src/main/java/org/apache/tajo/catalog/store/CatalogStore.java
 ---
    @@ -97,13 +97,31 @@ boolean existPartitionMethod(String databaseName, 
String tableName) throws Undef
        * @return
        * @throws TajoException
        */
    -  List<CatalogProtos.PartitionDescProto> getPartitions(String 
databaseName, String tableName) throws
    -      UndefinedDatabaseException, UndefinedTableException, 
UndefinedPartitionMethodException;
    +  List<CatalogProtos.PartitionDescProto> getAllPartitions(String 
databaseName, String tableName) throws
    +    UndefinedDatabaseException, UndefinedTableException, 
UndefinedPartitionMethodException,
    +    PartitionNotFoundException, UnsupportedException;
     
       CatalogProtos.PartitionDescProto getPartition(String databaseName, 
String tableName,
    -                                                String partitionName)
    -      throws UndefinedDatabaseException, UndefinedTableException, 
UndefinedPartitionException,
    -      UndefinedPartitionMethodException;
    +                                                String partitionName) 
throws UndefinedDatabaseException,
    +    UndefinedTableException, UndefinedPartitionMethodException, 
UndefinedPartitionException;
    +
    +  /**
    +   * PartitionedTableRewriter take a look into partition directories for 
rewriting filter conditions. But if there
    +   * are lots of sub directories on HDFS, such as, more than 10,000 
directories,
    +   * it might be cause overload to NameNode. Thus, CatalogStore need to 
provide partition directories for specified
    +   * filter conditions. This scan right partition directories on 
CatalogStore with where clause.
    +   *
    +   * @param request contains database name, table name, algebra 
expressions, parameter for executing PrepareStatement
    +   * @return list of TablePartitionProto
    +   * @throws UndefinedDatabaseException, UndefinedTableException, 
UndefinedPartitionMethodException
    +   */
    +  List<PartitionDescProto> getPartitionsByAlgebra(PartitionsByAlgebraProto 
request) throws
    +    UndefinedDatabaseException, UndefinedTableException, 
UndefinedPartitionMethodException,
    +    UndefinedOperatorException, PartitionNotFoundException, 
UnsupportedException;
    +
    +  List<PartitionDescProto> getPartitionsByFilter(PartitionsByFilterProto 
request) throws
    --- End diff --
    
    You need to add a description for this method.


> Make partition pruning based on catalog informations
> ----------------------------------------------------
>
>                 Key: TAJO-1493
>                 URL: https://issues.apache.org/jira/browse/TAJO-1493
>             Project: Tajo
>          Issue Type: Sub-task
>          Components: Catalog, Planner/Optimizer
>            Reporter: Jaehwa Jung
>            Assignee: Jaehwa Jung
>             Fix For: 0.11.0, 0.12.0
>
>         Attachments: TAJO-1493.patch, TAJO-1493_2.patch, TAJO-1493_3.patch, 
> TAJO-1493_4.patch
>
>
> Currently, PartitionedTableRewriter take a look into partition directories 
> for rewriting filter conditions. It get all sub directories of table path 
> because catalog doesn’t provide partition directories. But if there are lots 
> of sub directories on HDFS, such as, more than 10,000 directories, it might 
> be cause overload to NameNode. Thus, CatalogStore need to provide partition 
> directories for specified filter conditions. I designed new method to 
> CatalogStore as follows:
> * method name: getPartitionsWithConditionFilters
> * first parameter: database name
> * second parameter: table name
> * third parameter: where clause (included target column name and partition 
> value)
> * return values: 
> List<org.apache.tajo.catalog.proto.CatalogProtos.TablePartitionProto>
> * description: It scan right partition directories on CatalogStore with where 
> caluse. 
>   For examples, users set parameters as following:
> ** first parameter: default
> ** second parameter: table1
> ** third parameter: COLUMN_NAME = 'col1' AND PARTITION_VALUE = '3
> In the previous cases, this method will create select clause as follows.
> {code:xml}
> SELECT DISTINCT A.PATH
> FROM PARTITIONS A, (
>   SELECT B.PARTITION_ID
>   FROM PARTITION_KEYS B
>   WHERE B.PARTITION_ID > 0 
>   AND (
>     COLUMN_NAME = 'col1' AND PARTITION_VALUE = '3'
>   )
> ) B
> WHERE A.PARTITION_ID > 0
> AND A.TID = ${table_id}
> AND A.PARTITION_ID = B.PARTITION_ID
> {code}
> At the first time, I considered to use EvalNode instead of where clause. But 
> I can’t use it because of recursive related problems between tajo-catalog 
> module and tajo-plan module. So, I’ll implement utility class to convert 
> EvalNode to SQL.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to