[M] Change in asterixdb[lumina]: [NO ISSUE][EXT] Throw ExternalSourceError when colons are present in ...

[AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>]

From Hussain Towaileb <hu...@gmail.com>:

Attention is currently required from: Preetham Poluparthi.

Hussain Towaileb has posted comments on this change by Preetham Poluparthi. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20952?usp=email )

Change subject: [NO ISSUE][EXT] Throw ExternalSourceError when colons are present in HDFS path
......................................................................


Patch Set 3:

(1 comment)

File asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/HDFSDataSourceFactory.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20952/comment/53eac832_696f4efa?usp=email :
PS3, Line 235: LOGGER.info("This error can be due to presence of colons in the path "
             :                     + "of the files to be read. HDFS does not support colons in file paths. Please check the paths of the files.");
This goes to the log file, the user will not be aware of it. You can append or prepend this to the error message itself in the exception thrown so it is visible and the user can take action if that's the case.



-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20952?usp=email
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: asterixdb
Gerrit-Branch: lumina
Gerrit-Change-Id: Ifdb04b9df814dce7528d94339099f072265c8724
Gerrit-Change-Number: 20952
Gerrit-PatchSet: 3
Gerrit-Owner: Preetham Poluparthi <pr...@apache.org>
Gerrit-Reviewer: Hussain Towaileb <hu...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-CC: Anon. E. Moose #1000171
Gerrit-Attention: Preetham Poluparthi <pr...@apache.org>
Gerrit-Comment-Date: Thu, 02 Apr 2026 03:24:49 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No