[GitHub] [knox] adembo opened a new pull request #171: KNOX-2072: Kudu web UI service definition

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

adembo opened a new pull request #171: KNOX-2072: Kudu web UI service definition
URL: https://github.com/apache/knox/pull/171
 
 
   This patch adds a new service definition for accessing Apache Kudu's web UI.
   
   Like Apache Impala, every server has a web UI, so we use scheme/host/port
   query parameters to disambiguate between servers. Unlike Impala, the Kudu
   web UI will add /KNOX-BASE tokens to URLs in its web pages when it detects
   it is being proxied by Knox; we use these tokens as the basis for pattern
   matching in our rewrite rules.
   
   I tried to avoid cargo culting from existing service definitions and I added
   comments to explain syntax and other things that weren't obvious to me as a
   non-Knox developer.
   
   I tested this manually by deploying a small Kudu cluster on my machine along
   with a Knox gateway and a really simple topology. I then browsed every page
   in the web UI and verified its correct behavior.
   

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