smolnar82 opened a new pull request #280: KNOX-2265 - Checking CM configs by 
their related names and read hive.server2.use.SSL from the service configuration
URL: https://github.com/apache/knox/pull/280
 
 
   ## What changes were proposed in this pull request?
   
   CM configuration's related name is used when looking up service/role 
configurations.
   Moreover, it turned out that `hive.server2.use.SSL` is a service 
configuration rather than being a role configuration.
   
   ## How was this patch tested?
   
   Updated and ran JUnit tests and tested manually:
   
   1. had a secure remote cluster (i.e. SSL enabled) with Hive/Hive on TEZ
   2. redeployed Knox locally and added a descriptor with HIVE
   3. set the remote cluster's attributes in this descriptor's `Service 
Discovery` details
   4. confirmed that
     - service discovery worked
     - Hive URLs start with the `https` scheme

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

Reply via email to