Henry Robinson has posted comments on this change.
Change subject: IMPALA-4011: Remove / reword messages when statestore messages
Patch Set 1:
PS1, Line 630: // TODO: This should be a healthcheck in a monitored metric in
CM, which would
: // require a 'rate' metric type.
> remove this reference to CM while we're here
Ok, although bear in mind that CM references aren't verboten just because this
is an Apache project. Happy to remove here.
PS1, Line 633: "Missed subscriber ($0) $1 deadline by $2ms, "
: "consider increasing
--statestore_heartbeat_frequency_ms (currently $3) on "
: "this Statestore and
: "on subscribers",
> I'm worried it's too hard to know what value to pick. Do we have any guidan
I don't think there's any good advice out there, nor is there in the docs. At
some point I think we have to say "you figure it out" with these kinds of
warning messages, and it's not helpful necessarily to print a wall of text. A
good value is usually a larger one (as recommended), so users have a chance of
improving things because the amount by which deadlines were missed will go down
so there's a feedback loop.
PS1, Line 636: subscribers
> subscribers (Impala Daemons and the Catalog Server)
To view, visit http://gerrit.cloudera.org:8080/4500
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Owner: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Henry Robinson <he...@cloudera.com>
Gerrit-Reviewer: Matthew Jacobs <m...@cloudera.com>