opened
jwang47
username_0
CDAP-1230: HTTP endpoint to get effective configuration
issue
created
nitinmotgi
username_1
I had a version that I didn't merge and it was JSON response. The idea was that this API would be used in the CDAP Console. Can we `format=json|xml` if both formats are needed. Don't know the use-case for needing xml.
comment
created
jwang47
username_0
From the comments, looks like we want to expose HConf and CConf, and also allow for both json and xml formats.
comment
created
nitinmotgi
username_1
@username_0 it would also be good to have additional information about where the configuration was up picked from - Was it from cdap-default.xml or cdap-site.xml. That information is available +ve.
comment
created
nitinmotgi
username_1
@username_0 why do we need xml ?
comment
created
poornachandra
username_2
@username_1 If we want to duplicate configuration from a cluster to another, then it is easy if we have it in XML format.
comment
created
poornachandra
username_2
@username_0 `Configuration` class itself has methods to generate XML - `writeXml`. See if you can make use of it.

Also, if possible can you add the `ConfigHandler` to all HTTP system services like Transaction Manager, Explore, etc.?
comment
created
jwang47
username_0
Build running https://builds.cask.co/browse/CDAP-DUT594-1
comment
created
jwang47
username_0
https://builds.cask.co/browse/CDAP-DUT594-4
comment
created
anew
username_3
LGTM
comment