Github user twalthr commented on a diff in the pull request:
https://github.com/apache/flink/pull/5662#discussion_r173863375
--- Diff:
flink-connectors/flink-connector-kafka-base/src/test/java/org/apache/flink/streaming/connectors/kafka/KafkaJsonTableSourceFactoryTestBase.java
---
@@ -89,9 +94,10 @@ private void testTableSource(FormatDescriptor format) {
// construct table source using a builder
final Map<String, String> tableJsonMapping = new HashMap<>();
+ tableJsonMapping.put("name", "name");
--- End diff --
The question is what should a rowtime attribute field (or a custom
extractor) reference? The input or the current schema? I think it should
reference the input thus all fields (even the renamed ones) need to be present
in the mapping.
---
