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

Alexander Kolbasov commented on HIVE-18755:
-------------------------------------------

[~alangates] Thanks for your replies. A few more comments.

# Regarding optional vs non-optional. Your reasoning makes sense, but we will 
have some calls where catalogs must be optional (for compat reasons). As a 
result, for some calls it is optional and for some it is required which may be 
confusing - some calls need to check that the value is present or not, some 
don't and this will complicate things, so making them uniformly optional may be 
just simpler. If you think that's not an issue it is fine as well.
# {quote}On the comments about getCatalogs(), I don't plan to add a request 
structure as I don't envision there ever being a structure above catalogs 
(since that's the way SQL defines it).  {quote}. Here is an example use case - 
your proposed use of catalogs to distinguish between Hive catalog and, say, 
Kafka catalog. On top of that we want to add "cluster" namespace which 
distinguish what luster accesses the data and all namespaces may deal with hive 
and kafka catalogs. Even if you think that this isn't a valid use case now, it 
may become useful soon, so wouldn't it be better to avoid running into the same 
issues soon afterwards?
# {quote}As to getCatalogs returning a list of strings, it matches the name of 
getDatabases, which returns a list of DB names. {quote} This doesn't make 
getDatabases very good API either, so is it better to fix newer ones or 
duplicate old bad ones?
# On auto-creation your reasoning makes sense.
# For Location - I see what you mean, can you a) document that and b) make 
location optional since there are likely to be catalogs for which location 
isn't anyhow useful.


> Modifications to the metastore for catalogs
> -------------------------------------------
>
>                 Key: HIVE-18755
>                 URL: https://issues.apache.org/jira/browse/HIVE-18755
>             Project: Hive
>          Issue Type: Sub-task
>          Components: Metastore
>            Reporter: Alan Gates
>            Assignee: Alan Gates
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 3.0.0
>
>         Attachments: HIVE-18755.2.patch, HIVE-18755.3.patch, 
> HIVE-18755.nothrift, HIVE-18755.patch
>
>
> Step 1 of adding catalogs is to add support in the metastore.



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

Reply via email to