You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@helix.apache.org by zhan849 <gi...@git.apache.org> on 2018/10/31 21:08:33 UTC

[GitHub] helix pull request #283: [HELIX-775] consolidate user content related apis f...

GitHub user zhan849 opened a pull request:

    https://github.com/apache/helix/pull/283

    [HELIX-775] consolidate user content related apis for task driver

    HELIX-1315: consolidate user content related apis for task driver
    
    
    To consolidate task driver user content related apis, and corresponding rest apis, I'm deprecating the general getUserContent() api, but instead, we now have the following apis for get / add / update user content.
    
    ```java
    public void addOrUpdateWorkflowUserContentMap(String workflowName,
          final Map<String, String> contentToAddOrUpdate);
    
    public void addOrUpdateJobUserContentMap(String workflowName, String jobName,
          final Map<String, String> contentToAddOrUpdate);
    
    public void addOrUpdateTaskUserContentMap(String workflowName, String jobName,
          String taskPartitionId, final Map<String, String> contentToAddOrUpdate);
    
    
    public Map<String, String> getWorkflowUserContentMap(String workflowName);
    
    
    public Map<String, String> getJobUserContentMap(String workflowName, String jobName);
    
    public Map<String, String> getTaskUserContentMap(String workflowName, String jobName,
          String taskPartitionId);
    ```
    
    delete user content api tbd but can use the same convension

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

    $ git pull https://github.com/zhan849/helix harry/task-user-content

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

    https://github.com/apache/helix/pull/283.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 #283
    
----
commit b235c4ee5a82c5970d29e839317ea242813a58bc
Author: Harry Zhang <hr...@...>
Date:   2018-10-04T18:25:08Z

    [HELIX-775] consolidate user content related apis for task driver

----


---

[GitHub] helix pull request #283: [HELIX-775] consolidate user content related apis f...

Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:

    https://github.com/apache/helix/pull/283


---