[ 
https://issues.apache.org/jira/browse/BEAM-4196?focusedWorklogId=99488&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-99488
 ]

ASF GitHub Bot logged work on BEAM-4196:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 08/May/18 12:30
            Start Date: 08/May/18 12:30
    Worklog Time Spent: 10m 
      Work Description: kennknowles commented on a change in pull request 
#5276: [BEAM-4196][SQL] Support complex types in DDL
URL: https://github.com/apache/beam/pull/5276#discussion_r186710137
 
 

 ##########
 File path: 
sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/parser/SqlCreateTable.java
 ##########
 @@ -67,23 +72,24 @@
     this.tblProperties = tblProperties; // may be null
   }
 
+  @Override
   public List<SqlNode> getOperandList() {
-    return ImmutableNullableList.of(name, columnList, type, comment, location, 
tblProperties);
+    throw new UnsupportedOperationException(
+        "Getting operands CREATE TABLE is unsupported at the moment");
 
 Review comment:
   <!--new_thread; commit:4a38a26e2bfa0631ef5426455a57802eb81e5e93; 
resolved:1-->
   Just curious - where is it used? You can't really do much except copy the 
node if you don't know what kind of node it is, so just asking for my own 
education how Calcite (or BeamSQL) uses this.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


Issue Time Tracking
-------------------

    Worklog Id:     (was: 99488)
    Time Spent: 2h 40m  (was: 2.5h)

> [SQL] Support Complex Types in DDL
> ----------------------------------
>
>                 Key: BEAM-4196
>                 URL: https://issues.apache.org/jira/browse/BEAM-4196
>             Project: Beam
>          Issue Type: New Feature
>          Components: dsl-sql
>            Reporter: Anton Kedin
>            Assignee: Anton Kedin
>            Priority: Major
>          Time Spent: 2h 40m
>  Remaining Estimate: 0h
>
> Neither our DDL parser we copied from calcite-server or the calcite-server 
> don't support complex types in DDL. If we want to model something like JSON 
> objects we need to support at least Arrays and nested Rows.



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

Reply via email to