>From Murtadha Hubail <[email protected]>: Murtadha Hubail has submitted this change. ( https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19063 )
Change subject: [ASTERIXDB-3519][EXT]: Fix error message ...................................................................... [ASTERIXDB-3519][EXT]: Fix error message - user model changes: change error message - storage format changes: no - interface changes: no details: - Log the exception and show a different error message Change-Id: I97bdaf6639855158a6982e0015655c0e99795119 Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19063 Reviewed-by: Murtadha Hubail <[email protected]> Tested-by: Murtadha Hubail <[email protected]> --- M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java 1 file changed, 25 insertions(+), 2 deletions(-) Approvals: Murtadha Hubail: Looks good to me, approved; Verified Anon. E. Moose #1000171: diff --git a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java index 6d004d5..2984954 100644 --- a/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java +++ b/asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java @@ -89,10 +89,14 @@ import org.apache.hyracks.api.exceptions.Warning; import org.apache.hyracks.api.network.INetworkSecurityManager; import org.apache.hyracks.hdfs.scheduler.Scheduler; +import org.apache.logging.log4j.LogManager; +import org.apache.logging.log4j.Logger; import org.apache.parquet.hadoop.ParquetInputFormat; public class HDFSUtils { + private static final Logger LOGGER = LogManager.getLogger(); + private HDFSUtils() { } @@ -339,8 +343,8 @@ Files.writeString(krb5conf, String.format(ExternalDataConstants.KERBEROS_CONFIG_FILE_CONTENT, kerberosRealm, kerberosKdc)); } catch (IOException ex) { - throw AsterixException.create(ErrorCode.COULD_NOT_CREATE_FILE, - String.join("", ExternalDataConstants.KERBEROS_CONFIG_FILE_PATTERN)); + LOGGER.error("Failed to create kerberos config file", ex); + throw AsterixException.create(ErrorCode.COULD_NOT_CREATE_TOKENS); } return krb5conf; } -- To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/19063 To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Change-Id: I97bdaf6639855158a6982e0015655c0e99795119 Gerrit-Change-Number: 19063 Gerrit-PatchSet: 3 Gerrit-Owner: Savyasach Reddy <[email protected]> Gerrit-Reviewer: Anon. E. Moose #1000171 Gerrit-Reviewer: Murtadha Hubail <[email protected]> Gerrit-MessageType: merged
