You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@helix.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2018/10/31 20:57:00 UTC

[jira] [Commented] (HELIX-775) Task driver should support add/get task framework user content

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

ASF GitHub Bot commented on HELIX-775:
--------------------------------------

GitHub user zhan849 opened a pull request:

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

    [HELIX-775] add task driver support for helix rest to add/get task fr…

    …amework user content
    
    
    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);
    ```
    
    API for deleting user content is 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/282.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 #282
    
----
commit 7ec5313bccb679014d6a0605ee5d7184063e555e
Author: Harry Zhang <hr...@...>
Date:   2018-10-31T20:55:44Z

    [HELIX-775] add task driver support for helix rest to add/get task framework user content

----


> Task driver should support add/get task framework user content
> --------------------------------------------------------------
>
>                 Key: HELIX-775
>                 URL: https://issues.apache.org/jira/browse/HELIX-775
>             Project: Apache Helix
>          Issue Type: Task
>            Reporter: Harry Zhang
>            Assignee: Harry Zhang
>            Priority: Major
>
> Task driver should support add/get task framework user content at workflow/job/task levels
>  
> AC:
>  * finish implementation
>  * add tests



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)