Copilot commented on code in PR #9593:
URL: https://github.com/apache/seatunnel/pull/9593#discussion_r2227174863
##########
seatunnel-connectors-v2/connector-tdengine/src/main/java/org/apache/seatunnel/connectors/seatunnel/tdengine/config/TDengineSourceOptions.java:
##########
@@ -33,4 +35,21 @@ public class TDengineSourceOptions extends
TDengineCommonOptions {
.stringType()
.noDefaultValue()
.withDescription("The upper bound for data query range");
+
+ public static final Option<List<String>> SUB_TABLES =
+ Options.key("sub_tables")
+ .listType()
+ .noDefaultValue()
+ .withDescription(
+ "The sub table names to query data from, separated
by comma , "
+ + "if not specified, all sub tables will
be queried");
+
+ public static final Option<List<String>> READ_COLUMNS =
+ Options.key("read_columns")
+ .listType()
+ .noDefaultValue()
+ .withDescription(
+ "The field names to be written to TDengine "
+ + "If not specified, all fields will be
written. "
+ + "This option is useful when the source
schema does not match the TDengine table schema.");
Review Comment:
The description for READ_COLUMNS incorrectly mentions 'written to TDengine'
when it should describe reading/selecting columns from TDengine since this is a
source option.
```suggestion
"The column names to be read from TDengine. "
+ "If not specified, all columns will be
read. "
+ "This option is useful for selecting
specific columns when querying data from TDengine.");
```
--
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]