[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-10-03 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16190049#comment-16190049
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user asfgit closed the pull request at:

https://github.com/apache/beam/pull/3786


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-10-02 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16188754#comment-16188754
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user asfgit closed the pull request at:

https://github.com/apache/beam/pull/3912


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-27 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16183137#comment-16183137
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem opened a pull request:

https://github.com/apache/beam/pull/3912

[BEAM-2724] Updating BEAM_CONTAINER_VERSION for new worker

r: @charlesccychen 

Updating to a new worker harness that supports structured names for msec 
counters.

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

$ git pull https://github.com/pabloem/incubator-beam newworka

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

https://github.com/apache/beam/pull/3912.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 #3912


commit 66a988a9e1f97e13a162d004bc8fa6c2d871663d
Author: Pablo 
Date:   2017-09-27T19:44:35Z

Updating BEAM_CONTAINER_VERSION for new worker




> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-27 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16182782#comment-16182782
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user pabloem closed the pull request at:

https://github.com/apache/beam/pull/3786


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-27 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16182783#comment-16182783
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem reopened a pull request:

https://github.com/apache/beam/pull/3786

[BEAM-2724] Preparing statesampler to work with structured names




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

$ git pull https://github.com/pabloem/incubator-beam ssampler-structured

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

https://github.com/apache/beam/pull/3786.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 #3786






> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181834#comment-16181834
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user pabloem closed the pull request at:

https://github.com/apache/beam/pull/3786


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181835#comment-16181835
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem reopened a pull request:

https://github.com/apache/beam/pull/3786

[BEAM-2724] Preparing statesampler to work with structured names




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

$ git pull https://github.com/pabloem/incubator-beam ssampler-structured

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

https://github.com/apache/beam/pull/3786.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 #3786


commit 6897c0998bf14440f20ec6263d0e4efd2c81924a
Author: Pablo 
Date:   2017-08-29T22:23:20Z

Preparing statesampler to work with structured names

commit 9e6f3c00ac9ac908020cb9d9ad81ef1807c38ad7
Author: Pablo 
Date:   2017-09-21T15:57:00Z

Support iotarget for ssampler

commit b3d629fb27b7150b79f1bfcea973f5ff0c0a1b78
Author: Pablo 
Date:   2017-09-21T17:53:57Z

Improving documentation

commit b6e79645b5d6d2d356755f0d64f5244f9b3fa9e1
Author: Pablo 
Date:   2017-09-25T17:59:33Z

Fix missing arg

commit f10b3747bb7c869dab00ffc28d50b4cab4e238aa
Author: Pablo 
Date:   2017-09-25T22:47:21Z

Addressing comments

commit 932f5a1e632d7666f2e671095c603b69a0c823ec
Author: Pablo 
Date:   2017-09-25T23:13:54Z

Fix nit

commit 8cf66aa81d5f3fe0e6920db7050f65d2c05a3948
Author: Pablo 
Date:   2017-09-26T17:07:18Z

Fix typo




> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181748#comment-16181748
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user pabloem closed the pull request at:

https://github.com/apache/beam/pull/3786


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16181749#comment-16181749
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem reopened a pull request:

https://github.com/apache/beam/pull/3786

[BEAM-2724] Preparing statesampler to work with structured names




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

$ git pull https://github.com/pabloem/incubator-beam ssampler-structured

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

https://github.com/apache/beam/pull/3786.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 #3786


commit 6897c0998bf14440f20ec6263d0e4efd2c81924a
Author: Pablo 
Date:   2017-08-29T22:23:20Z

Preparing statesampler to work with structured names

commit 9e6f3c00ac9ac908020cb9d9ad81ef1807c38ad7
Author: Pablo 
Date:   2017-09-21T15:57:00Z

Support iotarget for ssampler

commit b3d629fb27b7150b79f1bfcea973f5ff0c0a1b78
Author: Pablo 
Date:   2017-09-21T17:53:57Z

Improving documentation

commit b6e79645b5d6d2d356755f0d64f5244f9b3fa9e1
Author: Pablo 
Date:   2017-09-25T17:59:33Z

Fix missing arg

commit f10b3747bb7c869dab00ffc28d50b4cab4e238aa
Author: Pablo 
Date:   2017-09-25T22:47:21Z

Addressing comments

commit 932f5a1e632d7666f2e671095c603b69a0c823ec
Author: Pablo 
Date:   2017-09-25T23:13:54Z

Fix nit

commit 8cf66aa81d5f3fe0e6920db7050f65d2c05a3948
Author: Pablo 
Date:   2017-09-26T17:07:18Z

Fix typo




> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16180328#comment-16180328
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user pabloem closed the pull request at:

https://github.com/apache/beam/pull/3786


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-26 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16180329#comment-16180329
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem reopened a pull request:

https://github.com/apache/beam/pull/3786

[BEAM-2724] Preparing statesampler to work with structured names




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

$ git pull https://github.com/pabloem/incubator-beam ssampler-structured

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

https://github.com/apache/beam/pull/3786.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 #3786


commit 6897c0998bf14440f20ec6263d0e4efd2c81924a
Author: Pablo 
Date:   2017-08-29T22:23:20Z

Preparing statesampler to work with structured names

commit 9e6f3c00ac9ac908020cb9d9ad81ef1807c38ad7
Author: Pablo 
Date:   2017-09-21T15:57:00Z

Support iotarget for ssampler

commit b3d629fb27b7150b79f1bfcea973f5ff0c0a1b78
Author: Pablo 
Date:   2017-09-21T17:53:57Z

Improving documentation

commit b6e79645b5d6d2d356755f0d64f5244f9b3fa9e1
Author: Pablo 
Date:   2017-09-25T17:59:33Z

Fix missing arg

commit f10b3747bb7c869dab00ffc28d50b4cab4e238aa
Author: Pablo 
Date:   2017-09-25T22:47:21Z

Addressing comments

commit 932f5a1e632d7666f2e671095c603b69a0c823ec
Author: Pablo 
Date:   2017-09-25T23:13:54Z

Fix nit




> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-25 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16180118#comment-16180118
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user pabloem closed the pull request at:

https://github.com/apache/beam/pull/3786


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-09-25 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16180119#comment-16180119
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem reopened a pull request:

https://github.com/apache/beam/pull/3786

[BEAM-2724] Preparing statesampler to work with structured names




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

$ git pull https://github.com/pabloem/incubator-beam ssampler-structured

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

https://github.com/apache/beam/pull/3786.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 #3786


commit 6897c0998bf14440f20ec6263d0e4efd2c81924a
Author: Pablo 
Date:   2017-08-29T22:23:20Z

Preparing statesampler to work with structured names

commit 9e6f3c00ac9ac908020cb9d9ad81ef1807c38ad7
Author: Pablo 
Date:   2017-09-21T15:57:00Z

Support iotarget for ssampler

commit b3d629fb27b7150b79f1bfcea973f5ff0c0a1b78
Author: Pablo 
Date:   2017-09-21T17:53:57Z

Improving documentation

commit b6e79645b5d6d2d356755f0d64f5244f9b3fa9e1
Author: Pablo 
Date:   2017-09-25T17:59:33Z

Fix missing arg

commit f10b3747bb7c869dab00ffc28d50b4cab4e238aa
Author: Pablo 
Date:   2017-09-25T22:47:21Z

Addressing comments

commit 932f5a1e632d7666f2e671095c603b69a0c823ec
Author: Pablo 
Date:   2017-09-25T23:13:54Z

Fix nit




> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-08-24 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16140292#comment-16140292
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user asfgit closed the pull request at:

https://github.com/apache/beam/pull/3747


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-08-22 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16137640#comment-16137640
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem opened a pull request:

https://github.com/apache/beam/pull/3747

[BEAM-2724] Support for IOTargetName like in updated IO dataflow metrics



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

$ git pull https://github.com/pabloem/incubator-beam add-iotnm

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

https://github.com/apache/beam/pull/3747.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 #3747


commit 7beb2abf8b9ed33882961ef9e993a3f53a9363df
Author: Pablo 
Date:   2017-08-22T23:53:32Z

Adding IOTargetName and unittests for CounterName




> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-08-09 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16120796#comment-16120796
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user asfgit closed the pull request at:

https://github.com/apache/beam/pull/3684


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-08-08 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16118736#comment-16118736
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem reopened a pull request:

https://github.com/apache/beam/pull/3684

[BEAM-2724] Preparing support for Structured Names in Dataflow counters

Follow this checklist to help us incorporate your contribution quickly and 
easily:

 - [x] Make sure there is a [JIRA 
issue](https://issues.apache.org/jira/projects/BEAM/issues/) filed for the 
change (usually before you start working on it).  Trivial changes like typos do 
not require a JIRA issue.  Your pull request should address just this issue, 
without pulling in other changes.
 - [x] Each commit in the pull request should have a meaningful subject 
line and body.
 - [x] Format the pull request title like `[BEAM-XXX] Fixes bug in 
ApproximateQuantiles`, where you replace `BEAM-XXX` with the appropriate JIRA 
issue.
 - [x] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
 - [x] Run `mvn clean verify` to make sure basic checks pass. A more 
thorough check will be performed on your pull request automatically.
---

* Adding a CounterName class to help support structured names for counters 
in the future.
* Also remove function calls from the `apiclient.py` file that are no 
longer used.
* And also removing function calls used for aggregators, which have been 
dead code for a while.

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

$ git pull https://github.com/pabloem/incubator-beam add-countername

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

https://github.com/apache/beam/pull/3684.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 #3684






> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-08-08 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16118735#comment-16118735
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

Github user pabloem closed the pull request at:

https://github.com/apache/beam/pull/3684


> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)


[jira] [Commented] (BEAM-2724) MSEC counters should support Structured Names in Dataflow

2017-08-03 Thread ASF GitHub Bot (JIRA)

[ 
https://issues.apache.org/jira/browse/BEAM-2724?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16113748#comment-16113748
 ] 

ASF GitHub Bot commented on BEAM-2724:
--

GitHub user pabloem opened a pull request:

https://github.com/apache/beam/pull/3684

[BEAM-2724] Preparing support for Structured Names in Dataflow counters

Follow this checklist to help us incorporate your contribution quickly and 
easily:

 - [x] Make sure there is a [JIRA 
issue](https://issues.apache.org/jira/projects/BEAM/issues/) filed for the 
change (usually before you start working on it).  Trivial changes like typos do 
not require a JIRA issue.  Your pull request should address just this issue, 
without pulling in other changes.
 - [x] Each commit in the pull request should have a meaningful subject 
line and body.
 - [x] Format the pull request title like `[BEAM-XXX] Fixes bug in 
ApproximateQuantiles`, where you replace `BEAM-XXX` with the appropriate JIRA 
issue.
 - [x] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
 - [x] Run `mvn clean verify` to make sure basic checks pass. A more 
thorough check will be performed on your pull request automatically.
---

Adding a CounterName class to help support structured names for counters in 
the future.
Also remove function calls from the `apiclient.py` file that are no longer 
used.

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

$ git pull https://github.com/pabloem/incubator-beam add-countername

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

https://github.com/apache/beam/pull/3684.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 #3684


commit b1e15abc0423ea7c8013f3d745ab97b44c04b5f5
Author: Pablo 
Date:   2017-08-04T00:46:09Z

Preparing support for Structured Names in Dataflow counters




> MSEC counters should support Structured Names in Dataflow
> -
>
> Key: BEAM-2724
> URL: https://issues.apache.org/jira/browse/BEAM-2724
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)