Henry Robinson has posted comments on this change. Change subject: IMPALA-4278: Don't abort Catalog startup quickly if HMS is not present ......................................................................
Patch Set 2: > Thanks!Do you think I should add some code to catalog to validate the > value of 'initial_hms_cnxn_timeout_s'? What would be the acceptable > range? You could validate that it was > 0, but otherwise I don't think you should put a bound on it. Users may want to wait effectively forever, and I can see them setting 9999999999 as a value to simulate that. > Adding this config parameter to the CM UI is another issue. Do you think it > should be added for the 5.10 release or it is not that urgent? I guess, the > default value is reasonable for most users and they can always set it as a > safety valve if they have to. Sounds like you're talking about a Cloudera release. Reminder that this is the Apache Impala project - vendor specific concerns should be discussed elsewhere. -- To view, visit http://gerrit.cloudera.org:8080/5095 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I546d8fe9836004832ae40110c9fe22b3e704e11b Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Attila Jeges <[email protected]> Gerrit-Reviewer: Attila Jeges <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-HasComments: No
