You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nifi.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/04/08 14:20:25 UTC
[jira] [Commented] (NIFI-1727) Refactor NCM Node Response merging
[ https://issues.apache.org/jira/browse/NIFI-1727?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15232109#comment-15232109 ]
ASF GitHub Bot commented on NIFI-1727:
--------------------------------------
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/338
NIFI-1727: Refactored logic for merging HTTP Requests
Refactored logic for merging HTTP Requests that are federated across cluster
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/markap14/nifi NIFI-1727
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/nifi/pull/338.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 #338
----
commit 42d141fe5c5bc438e3cbd8d30c9d13f1b9e5e25b
Author: Mark Payne <ma...@hotmail.com>
Date: 2016-04-08T12:17:11Z
NIFI-1727: Refactored logic for merging HTTP Requests that are federated across cluster
----
> Refactor NCM Node Response merging
> ----------------------------------
>
> Key: NIFI-1727
> URL: https://issues.apache.org/jira/browse/NIFI-1727
> Project: Apache NiFi
> Issue Type: Improvement
> Components: Core Framework
> Reporter: Mark Payne
> Assignee: Mark Payne
> Fix For: 1.0.0
>
>
> Currently, when the NCM federates a request to nodes in the cluster, those responses are merged using one massive 'mergeResponses' method, which delegates to many different 'mergeXYZ' methods. The logic is very repetitive and tedious to update and lives within the WebClusterManager.
> We need to simplify the logic, since it is very repetitive and need to pull the logic out of WebClusterManager into a separate interface so that the code can be cleanly pulled from the NCM and merged into the cluster coordinator on the nodes.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)