Adar Dembo has uploaded a new change for review.
http://gerrit.cloudera.org:8080/5435
Change subject: csd: two minor changes
......................................................................
csd: two minor changes
Revert the memory limit scaleFactor to the default value (1.0). 1.3 is
recommended for Java processes; not sure why Kudu was using that.
Convert the core dump directory to two separate role-level parameters. I
relented and did this once I saw that Impala also defines its core dump
directory in this way. In testing, the conversion was fine when the
service-level parameter wasn't set. When it was set, I saw that the
overridden value carried over to the roles even after the conversion
(good!). However, attempting to remove the so-called "role" overrides in the
UI yielded an error (bad!).
/api/v8/clusters/Cluster%201/services/KUDU-1/roles/KUDU-KUDU_MASTER-1/config?message=Updated%20service%20and%20role%20type%20configurations.
\
{ "message" : "Could not find config to delete with template name:
core_dump_directory" }
So this probably makes the conversion a non-starter, right?
Change-Id: I74b2fcb69fde9e0dbb232dba78db53cc1543b489
---
M java/kudu-csd/src/descriptor/service.sdl
1 file changed, 19 insertions(+), 11 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/35/5435/1
--
To view, visit http://gerrit.cloudera.org:8080/5435
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I74b2fcb69fde9e0dbb232dba78db53cc1543b489
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <[email protected]>
