anujmodi2021 commented on code in PR #6879:
URL: https://github.com/apache/hadoop/pull/6879#discussion_r1696680906
##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystem.java:
##########
@@ -213,6 +217,23 @@ public void initialize(URI uri, Configuration
configuration)
TracingContext tracingContext = new TracingContext(clientCorrelationId,
fileSystemId, FSOperationType.CREATE_FILESYSTEM,
tracingHeaderFormat, listener);
+
+ /*
+ * Validate the service type configured in the URI is valid for account
type used.
+ * HNS Account Cannot have Blob Endpoint URI.
+ */
+ try {
+ abfsConfiguration.validateConfiguredServiceType(
+ tryGetIsNamespaceEnabled(new TracingContext(tracingContext)));
+ } catch (InvalidConfigurationValueException ex) {
+ LOG.debug("File system configured with Invalid Service Type", ex);
+ throw ex;
+ } catch (AzureBlobFileSystemException ex) {
+ LOG.debug("Enable to determine account type for service type
validation", ex);
Review Comment:
Thanks for the suggestion, Added the test
`testFileSystemInitFailsIfNotAbleToDetermineAccountType` in class
ITestAzureBlobFileSystemInitAndCreate
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
