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

Eugene Koifman commented on HIVE-20556:
---------------------------------------

[~jmarhuen] could you create RB for this?
{{optional i64 id}} would it be better to make it required and give it a 
default value, like -1?
It seems odd that this is optional.

I see {{tbl.unsetId()}} in a number of places. Could you explain what it's for?

if this a read-only field, should MTable.setId(long id) be public?



> Expose an API to retrieve the TBL_ID from TBLS in the metastore tables
> ----------------------------------------------------------------------
>
>                 Key: HIVE-20556
>                 URL: https://issues.apache.org/jira/browse/HIVE-20556
>             Project: Hive
>          Issue Type: New Feature
>          Components: Metastore, Standalone Metastore
>            Reporter: Jaume M
>            Assignee: Jaume M
>            Priority: Major
>         Attachments: HIVE-20556.1.patch, HIVE-20556.10.patch, 
> HIVE-20556.11.patch, HIVE-20556.12.patch, HIVE-20556.13.patch, 
> HIVE-20556.14.patch, HIVE-20556.15.patch, HIVE-20556.2.patch, 
> HIVE-20556.3.patch, HIVE-20556.4.patch, HIVE-20556.5.patch, 
> HIVE-20556.6.patch, HIVE-20556.7.patch, HIVE-20556.8.patch, HIVE-20556.9.patch
>
>
> We have two options to do this
> 1) Use the current MTable and add a field for this value
> 2) Add an independent API call to the metastore that would return the TBL_ID.
> Option 1 is preferable.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to