manika137 commented on code in PR #8152:
URL: https://github.com/apache/hadoop/pull/8152#discussion_r2667526088


##########
hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/AzureBlobFileSystemStore.java:
##########
@@ -798,9 +798,12 @@ private AbfsOutputStreamContext 
populateAbfsOutputStreamContext(
       ContextEncryptionAdapter contextEncryptionAdapter,
       TracingContext tracingContext) {
     int bufferSize = abfsConfiguration.getWriteBufferSize();
+    getClientHandler().initServiceType(abfsConfiguration);

Review Comment:
   only reqd for testing purpose. We have tests where we validate ingress 
handlers based on configurations. Now since we set ingress service type from 
clienthandler instead of abfsconfig directly- we need to reassign it.
   For user flow- it still remains the same since config wont change in the 
middle of workload



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to