ferenc-csaky commented on code in PR #234:
URL:
https://github.com/apache/flink-connector-aws/pull/234#discussion_r3305862761
##########
flink-connector-aws/flink-connector-dynamodb/src/test/resources/create-table.sql:
##########
@@ -15,6 +15,8 @@ CREATE TABLE dynamo_db_table (
`some_time` TIME,
`some_timestamp` TIMESTAMP(3),
`some_timestamp_ltz` TIMESTAMP_LTZ(5),
+ `some_row` ROW<myField INT, myOtherField String>,
+ `some_row_in_row` ROW<myField INT, ROW<innerRowField INT,
innerRowOtherField String>>,
Review Comment:
The test fails, cause this col definition is invalid. I think what we miss
is the field name for the inner row, so it should be:
```SQL
`some_row_in_row` ROW<myField INT, myInnerRow ROW<innerRowField INT,
innerRowOtherField STRING>>
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
