Ivan-gfan commented on code in PR #5420:
URL: https://github.com/apache/seatunnel/pull/5420#discussion_r1314370462
##########
seatunnel-connectors-v2/connector-file/connector-file-sftp/src/main/java/org/apache/seatunnel/connectors/seatunnel/file/sftp/source/SftpFileSource.java:
##########
@@ -72,7 +74,7 @@ public void prepare(Config pluginConfig) throws
PrepareFailException {
if (fileFormat == FileFormat.ORC || fileFormat == FileFormat.PARQUET) {
throw new FileConnectorException(
CommonErrorCode.ILLEGAL_ARGUMENT,
- "Sftp file source connector only support read [text, csv,
json] files");
+ "Sftp file source connector does not support read [orc,
parquet] files");
Review Comment:
I thought it was more in line with the intent of the code cause I added the
`xml` file type, so I changed here, should I change back? Just like `Sftp file
source connector only support read [text, csv, json, (xml)] files);`
--
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]
