[jira] [Commented] (DRILL-7672) Make metadata type required when reading from / writing into Drill Metastore

["ASF GitHub Bot (Jira)" <ji...@apache.org>]

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

ASF GitHub Bot commented on DRILL-7672:
---------------------------------------

arina-ielchiieva commented on pull request #2042: DRILL-7672: Make metadata type required when reading from / writing into Drill Metastore
URL: https://github.com/apache/drill/pull/2042
 
 
   # [DRILL-7672](https://issues.apache.org/jira/browse/DRILL-7672): Make metadata type required when reading from / writing into Drill Metastore
   
   ## Description
   
   1. Upgraded Iceberg version and removed unneeded code for In / NotIn Expressions.
   2. Updated Metastore Read / Write interfaces to support required metadata types:
    a. introduced abstract Read / Write classes with boilerplate code;
    b. added delete operation with filter and metadata type;
    c. added metadata type validator which checks supported metadata types for each component;
    d. made purge operation terminal;
    e. made necessary changes in REAME.md files.
   3. Added / updated unit tests.
   
   ## Documentation
   NA
   
   ## Testing
   Added unit tests.
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Make metadata type required when reading from / writing into Drill Metastore
> ----------------------------------------------------------------------------
>
>                 Key: DRILL-7672
>                 URL: https://issues.apache.org/jira/browse/DRILL-7672
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.17.0
>            Reporter: Arina Ielchiieva
>            Assignee: Arina Ielchiieva
>            Priority: Major
>             Fix For: 1.18.0
>
>
> Metastore consists of components: TABLES, VIEWS etc (so far only TABLES are implemented). Each component metadata can have types. For examples, TABLES metadata can be of the following types: TABLE, SEGMENT, FILE, ROW_GROUP, PARTITION.
> During initial Metastore implementation when reading from / writing into Metastore, metadata type was indicated in filter expressions. 
> For Iceberg Metastore where all data is stored in files this was not this critical, basically when information is retrieved about the table, table folder is queried.
> For other Metastore implementations knowing metadata type can be more critical. For example, RDBMS Metastore would store TABLES metadata in different tables thus knowing which table to query would improve performance rather than trying to query all tables.
> Of course, we could traverse query filter and look for the hints which metadata type is needed but it is much better to know required metadata type beforehand without any extra logic.
> Taking into account that Metastore metadata is queried only in Drill code, developer knows beforehand what he needs to get / update / delete.
> This Jira aims to make metadata type required when reading from / writing into Drill Metastore. This change does not have any affect on the users, just internal code refactoring.
>  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)