nrg4878 commented on a change in pull request #2037:
URL: https://github.com/apache/hive/pull/2037#discussion_r606268405



##########
File path: 
ql/src/java/org/apache/hadoop/hive/ql/ddl/database/create/CreateDatabaseAnalyzer.java
##########
@@ -70,19 +73,41 @@ public void analyzeInternal(ASTNode root) throws 
SemanticException {
         managedLocationUri = 
unescapeSQLString(childNode.getChild(0).getText());
         outputs.add(toWriteEntity(managedLocationUri));
         break;
+      case HiveParser.TOK_DATACONNECTOR:
+        type = DatabaseType.REMOTE.name();
+        ASTNode nextNode = (ASTNode) root.getChild(i);
+        connectorName = ((ASTNode)nextNode).getChild(0).getText();
+        outputs.add(toWriteEntity(connectorName));
+        if (managedLocationUri != null) {

Review comment:
       so any "create table" from within a REMOTE database, will be invoked on 
the connector. Currently, we do not support any write DDLs in remote databases. 
The goal is to implement some of these going forward. no immediate need.
   But to answer your question, we do not support having local hive tables 
within REMOTE databases.
   
   This is from AbstractDataConnectorProvider 
     @Override
     public boolean createTable(Table table) throws MetaException {
       throw new MetaException("Creation of table in remote datasource is not 
supported.");
     }
   
     @Override
     public boolean dropTable(String tableName) throws MetaException {
       throw new MetaException("Deletion of table in remote datasource is not 
supported.");
     }
   
     @Override
     public boolean alterTable(String tableName, Table table) throws 
MetaException {
       throw new MetaException("Alter table in remote datasource is not 
supported.");
     }




-- 
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:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to