GitHub user rmatharu opened a pull request:
https://github.com/apache/samza/pull/897
SEP-19 : Refactoring to remove localityManager from JobModel
This PR removes localityManager from JobModel.
There was some localityMapping creation location in JobModel that was based
on localityManager being null.
That has, for now, been moved to JobModelManager.
A subsequent PR will completely get rid of that null-based logic.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rmatharu/samza jobmodelref
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/samza/pull/897.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 #897
----
commit 2c09b081d17848a357c0a3a8ecafbe36da6abd39
Author: Ray Matharu <rmatharu@...>
Date: 2019-01-24T02:16:05Z
Rocksdb bug fix
commit 916f66aed5afb7893629f3d2c15e435daaa5f71e
Author: Ray Matharu <rmatharu@...>
Date: 2019-01-25T00:58:18Z
Merge branch 'master' of https://github.com/apache/samza
commit 08006871900fe4b992c383f10fcaf5f7f75e5dcd
Author: Ray Matharu <rmatharu@...>
Date: 2019-01-25T19:34:28Z
Merge branch 'master' of https://github.com/apache/samza
commit c4e52eec08ed844f34b9196b3b10fae55f0ad674
Author: Ray Matharu <rmatharu@...>
Date: 2019-01-26T00:48:00Z
SEP-19 : Refactoring to remove localityManager from JobModel
----
---
