You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/03/29 22:17:41 UTC

[jira] [Commented] (SAMZA-1175) Removing CoordinationUtils from JobCoordinationFactory interface

    [ https://issues.apache.org/jira/browse/SAMZA-1175?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15947991#comment-15947991 ] 

ASF GitHub Bot commented on SAMZA-1175:
---------------------------------------

GitHub user navina opened a pull request:

    https://github.com/apache/samza/pull/102

    SAMZA-1175 - Removing CoordinationService from JobCoordinatorFactory interface

    @sborya I have made the changes. Please review. Thanks!

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/navina/samza SAMZA-1175

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/samza/pull/102.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 #102
    
----
commit b0b31979ab5da4d83878152058f446c42dbce82f
Author: navina <na...@apache.org>
Date:   2017-03-29T22:09:05Z

    Removing CoordinationService from JobCoordinatorFactory interface

----


> Removing CoordinationUtils from JobCoordinationFactory interface
> ----------------------------------------------------------------
>
>                 Key: SAMZA-1175
>                 URL: https://issues.apache.org/jira/browse/SAMZA-1175
>             Project: Samza
>          Issue Type: Bug
>            Reporter: Navina Ramesh
>            Assignee: Navina Ramesh
>
> SAMZA-1151 introduces a CoordinationServiceFactory that creates a CoordinationUtils instance. This does not have to break the interface contract in JobCoordinatorFactory. Rather, it isn't clear why this interface was changed initially. Hence, undoing that change until there is further clarity on it.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)