-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat(hue): enable application log on jobbrowser of hue #800
Conversation
I'm wondering of the implications of adding additional extras components to Additionally, I was evaluating the possibility of having a dedicated overrides directory for |
We cannot add a value to a string with the current override mechanism. This string would have to be a dict if we wanted to do it. |
I was leaning more towards the idea of overriding the entire key using an |
I think we should not duplicate a variable from If Hue is not deployed in the cluster, I think it is not a concern for it to be present in the configuration in The alternative would be to change these variables to dict but I don't think we gain anything there. |
IMO we gain for the-collection-extras, or any other community collection, to be independent from tdp-collections, as more and more external variables might be declared in tdp-collection. But that's about it agreed. |
I agree to make dicts but how can we know that certain variable must be a dict? Do we look for all the variables to put them in dict or do we change them to dict only if necessary? |
As discuss, will be done in #801. |
Which issue(s) this PR fixes
Give hue user access to yarn and spark application log
Additional comments
Agreements