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

2018-10-31 Thread Hudson (JIRA)


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

Hudson commented on HELIX-775:
--

FAILURE: Integrated in Jenkins build helix #1560 (See 
[https://builds.apache.org/job/helix/1560/])
[HELIX-775] consolidate user content related apis for task driver (hrzhang: rev 
b235c4ee5a82c5970d29e839317ea242813a58bc)
* (edit) helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
* (edit) helix-core/src/main/java/org/apache/helix/task/TaskDriver.java
* (edit) 
helix-core/src/test/java/org/apache/helix/task/TestGetSetUserContentStore.java


> 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)


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

2018-10-31 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on HELIX-775:
--

Github user asfgit closed the pull request at:

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


> 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)


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

2018-10-31 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on HELIX-775:
--

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 contentToAddOrUpdate);

public void addOrUpdateJobUserContentMap(String workflowName, String 
jobName,
  final Map contentToAddOrUpdate);

public void addOrUpdateTaskUserContentMap(String workflowName, String 
jobName,
  String taskPartitionId, final Map 
contentToAddOrUpdate);


public Map getWorkflowUserContentMap(String workflowName);


public Map getJobUserContentMap(String workflowName, String 
jobName);

public Map 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 
Date:   2018-10-04T18:25:08Z

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




> 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)


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

2018-10-31 Thread Hudson (JIRA)


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

Hudson commented on HELIX-775:
--

FAILURE: Integrated in Jenkins build helix #1559 (See 
[https://builds.apache.org/job/helix/1559/])
[HELIX-775] add task driver support for helix rest to add/get task (hrzhang: 
rev 7ec5313bccb679014d6a0605ee5d7184063e555e)
* (edit) helix-core/src/main/java/org/apache/helix/task/TaskUtil.java
* (edit) helix-core/src/main/java/org/apache/helix/task/TaskDriver.java


> 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)


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

2018-10-31 Thread ASF GitHub Bot (JIRA)


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

ASF GitHub Bot commented on HELIX-775:
--

Github user asfgit closed the pull request at:

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


> 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)


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

2018-10-31 Thread ASF GitHub Bot (JIRA)


[ 
https://issues.apache.org/jira/browse/HELIX-775?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=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 contentToAddOrUpdate);

public void addOrUpdateJobUserContentMap(String workflowName, String 
jobName,
  final Map contentToAddOrUpdate);

public void addOrUpdateTaskUserContentMap(String workflowName, String 
jobName,
  String taskPartitionId, final Map 
contentToAddOrUpdate);


public Map getWorkflowUserContentMap(String workflowName);


public Map getJobUserContentMap(String workflowName, String 
jobName);

public Map 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 
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)