[GitHub] [hadoop] bharatviswa504 edited a comment on issue #930: HDDS-1651. Create a http.policy config for Ozone

[GitBox <gi...@apache.org>]

bharatviswa504 edited a comment on issue #930: HDDS-1651. Create a http.policy config for Ozone
URL: https://github.com/apache/hadoop/pull/930#issuecomment-505229785
 
 
   Hi,
   I have a question:
   What is the reason to create a new http policy for ozone? Because when https is enabled, some of the additional config like key store location we still use dfs.https.server.keystore.resource. 
   
   I feel ozone can also re-use the hdfs config. In this way, we don't miss the code changes where HTTP policy is being used, like OzoneManagerSnapShotProvider which uses still DFSUtils.getHttpPolicy.
   
   And also we use to create HttpServer2.Builder builder with below code. That again uses dfs.http.policy. So, this is the reason we need to set ozone.http.policy to dfs.http.policy to make it work. So, I feel to avoid all these, we can use dfs.http.policy as before. Let me know your thoughts on this.
   ```
         builder = DFSUtil.httpServerTemplateForNNAndJN(conf, this.httpAddress,
             this.httpsAddress, name, getSpnegoPrincipal(), getKeytabFile());
   ```

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org