You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@knox.apache.org by "smolnar82 (via GitHub)" <gi...@apache.org> on 2023/04/18 08:54:01 UTC

[GitHub] [knox] smolnar82 opened a new pull request, #749: KNOX-2899 - Disable service-based discovery filter in Cloudera Manager service discovery

smolnar82 opened a new pull request, #749:
URL: https://github.com/apache/knox/pull/749

   ## What changes were proposed in this pull request?
   
   Service-based filtering is temporarily disabled while discovering a CM cluster.
   
   ## How was this patch tested?
   
   Manually tested using a large CM cluster with lots of services.
   


-- 
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: dev-unsubscribe@knox.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [knox] smolnar82 commented on pull request #749: KNOX-2899 - Disable service-based discovery filter in Cloudera Manager service discovery

Posted by "smolnar82 (via GitHub)" <gi...@apache.org>.
smolnar82 commented on PR #749:
URL: https://github.com/apache/knox/pull/749#issuecomment-1512906616

   For reviewers: I wanted to do the least intrusive change possible, that's why the private method and the `includedServices` parameter are still in place and in use. Since everything happens in memory, it's not a real performance issue.


-- 
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: dev-unsubscribe@knox.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [knox] smolnar82 merged pull request #749: KNOX-2899 - Disable service-based discovery filter in Cloudera Manager service discovery

Posted by "smolnar82 (via GitHub)" <gi...@apache.org>.
smolnar82 merged PR #749:
URL: https://github.com/apache/knox/pull/749


-- 
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: dev-unsubscribe@knox.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org