GitHub user narendly opened a pull request:
https://github.com/apache/helix/pull/271
[HELIX-756] TASK: Change LOG mode from info to debug
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/narendly/helix master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/helix/pull/271.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #271
----
commit 5140db0c50439c115d0c7d2637f7ad723f6f147a
Author: Hunter Lee <hulee@...>
Date: 2018-09-25T17:19:31Z
[HELIX-754] TASK: Fix LiveInstanceCurrentState change flag
Previously, existsLiveInstanceOrCurrentStateChange was getting reset in
ClusterDataCache when its getter was called. This was problematic because if
there were multiple jobs or multiple workflows, whoever calls this getter would
get the correct flag value, and the ensuing callers would get a false because
the flag would have been reset. This RB fixes that bug by reseting the flat
right in the beginning of refresh() call in ClusterDataCache, which allows all
callers during that pipeline would get the same, correct value.
Changelist:
1. Change the getter so that it does not reset the flag; instead, reset the
flag in the beginning of refresh()
commit e9f6c98dc58cb7bb842ff1f41063174003277823
Author: Hunter Lee <hulee@...>
Date: 2018-09-25T17:22:36Z
[HELIX-755] TASK: Build quota profile from scratch every rebalance
It has been reported that instances have a full quota despite no tasks
existing in their CURRENTSTATES. The cause of this is not clear, so making
ClusterDataCache trigger a refresh of all AssignableInstances will ensure that
there aren't situations where it looks like there has been a thread leak.
Optimizations will be implemented if necessary. Changelist: 1. Make
AssignableInstanceManager build all AssignableInstances from scratch every
rebalance
commit 4aaa00727fe34b9cdfde2978eeb8a892dcf29add
Author: Hunter Lee <hulee@...>
Date: 2018-09-25T17:25:39Z
[HELIX-756] TASK: Change LOG mode from info to debug
In production, it was observed that some users were running thousands of
tasks, and since AssignableInstance leaves a line of log for each task assigned
or released, the amount of log that was being generated was too much, and it
was too verbose.
Changelist:
1. Change the logging mode from info to debug in AssignableInstance and
AssignableInstanceManager
----
---
