[GitHub] incubator-beam pull request #1526: [BEAM-1101] Remove inconsistencies in Pyt...

2016-12-19 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1526

[BEAM-1101] Remove inconsistencies in Python PipelineOptions

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit 6f091898a84154d63974f4fa0717406f472d99a3
Author: Pablo <pabl...@google.com>
Date:   2016-12-07T02:01:54Z

Fixing inconsistencies in PipelineOptions




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1526: [BEAM-1101] Remove inconsistencies in Pyt...

2016-12-19 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1526


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1443: [BEAM-1055] Display Data keys on Python a...

2016-12-07 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1443


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1526: [BEAM-1101] Remove inconsistencies in Pyt...

2016-12-06 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1526

[BEAM-1101] Remove inconsistencies in Python PipelineOptions

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit c6362aed50eae9cf8b9e5ec6d0885132f3278e32
Author: Pablo <pabl...@google.com>
Date:   2016-12-07T02:01:54Z

Fixing inconsistencies in PipelineOptions




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1443: [BEAM-1055] Display Data keys on Python a...

2016-12-05 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1443

[BEAM-1055] Display Data keys on Python are inconsistent

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit d58d4cafa0a7fef9de59ee473d1262612bcf8694
Author: Pablo <pabl...@google.com>
Date:   2016-11-28T21:05:47Z

Giving all display data keys in Python a standard snake_case format.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1443: [BEAM-1055] Display Data keys on Python a...

2016-12-05 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1443


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1456: [BEAM-1046] - Travis tests on Python fail...

2016-12-05 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1456


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1395: Fixing error with PipelineOptions Display...

2016-12-01 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1395


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1456: Testing travis python issue

2016-11-29 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1456

Testing travis python issue

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---



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

$ git pull https://github.com/pabloem/incubator-beam test-tox-python

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

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


commit 24f133e9e4c675497b1cb3e9db672db51db180da
Author: Pablo <pabl...@google.com>
Date:   2016-11-29T21:58:55Z

Testing travis python issue




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1455: Exploring travis issue in Python SDK's li...

2016-11-29 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1455


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1455: Exploring travis issue in Python SDK's li...

2016-11-29 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1455

Exploring travis issue in Python SDK's linux runs

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

$ git pull https://github.com/pabloem/incubator-beam test-tox-python

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

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


commit 009230e8b6e58f2e79d5c6d9a24cb00531d87735
Author: Dan Halperin <dhalp...@google.com>
Date:   2016-09-09T20:12:27Z

Closes #933

commit acf530e11d90dc586b5b5f822746ef1bca0ac77c
Author: Charles Chen <c...@google.com>
Date:   2016-09-09T21:07:00Z

Use cStringIO instead of StringIO

commit 2649372d6ef10255366e34ee1fcf2f64766b4fd0
Author: Dan Halperin <dhalp...@google.com>
Date:   2016-09-10T19:39:13Z

Closes #939

commit 6a483c187dff5d436736f1ff505187c727a95b38
Author: Gus Katsiapis <katsia...@katsiapis-linux.mtv.corp.google.com>
Date:   2016-09-09T22:34:39Z

Making Dataflow Python Materialized PCollection representation more
efficient (3 of several).

commit bc32bc8661b31036a879cb2b569b1fe976e2ac4b
Author: Dan Halperin <dhalp...@google.com>
Date:   2016-09-13T00:07:22Z

Closes #940

commit f5557c00cd46c8560cad5027678603056a6235ff
Author: Gus Katsiapis <katsia...@katsiapis-linux.mtv.corp.google.com>
Date:   2016-09-12T17:11:44Z

Refactoring code in avroio.py to allow for re-use.

* Making sure that _AvroUtils validates the sync_marker.
* This should detect corrupted or not-properly formatted AVRO files.
* Simplifying block reading.
* Running snappy tests only when snappy is installed in the system.

commit 4b584ca26b12f42a21b25b7d72c2e57b20a1e7ad
Author: Dan Halperin <dhalp...@google.com>
Date:   2016-09-13T16:59:56Z

Closes #946

commit 4602c954caeda24cc7ce762c2fd9995aa4c04c1a
Author: Charles Chen <c...@google.com>
Date:   2016-09-14T19:14:34Z

Insert a shuffle before write finalization

commit a60b58a947fda65cb2f6d8c0bcbbf0756c8407bf
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-09-14T23:41:14Z

Closes #958

commit 63904e01f52be498ba3c8c9e1989bcffb02dfcd6
Author: Robert Bradshaw <rober...@gmail.com>
Date:   2016-09-16T00:00:42Z

Allow pickling of UnwindowedValues instances

These are trivial wrappers produced when "windowing" by the global window, 
but may need to be materialized in some cases.

commit e9b1e41240d5032cdaa2b745c95f94da45475f34
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-09-16T22:41:22Z

Add unit test for unwindowed iterator picking.

Also lifted this out to a top-level class rather than defining
it on every element now that it's not longer a simple generator
statement.

commit ebae225ed30854296862c46c497d4b3e6e7de9f6
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-09-16T22:50:09Z

Closes #964

commit 2d1e7ff6d342442f83b97c382f08a03e2bac6572
Author: Chamikara Jayalath <chamik...@google.com>
Date:   2016-08-30T01:08:46Z

Adds a text source to Python SDK.

commit 4b7fe2dc552c838c00e062a644d7d4bc507cf075
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-09-17T01:01:47Z

Closes #920

commit 24bb8f19329b3d0c1d0330e0c16c41ab1554684d
Author: Marian Dvorsky <mari...@google.com>
Date:   2016-09-16T17:46:32Z

Removed unnecessary throttling of rename parallelism.

commit 29b55e95600fbf299ac7eb9527ff33bd8030275e
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-09-20T00:39:47Z

Closes #965

commit 7de9830d96c7928444a5a4849698e70ec423ef62
Author: Christian Hudon <chr...@pianocktail.org>
Date:   2016-09-15T19:10:57Z

Use sys.executable and "-m pip" to ensure we use the same Python and pip as 
the currently running one.

commit 2f09003e33b63594fd87a6cd8bf2803005174fd1
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-09-20T00:50:58Z

Closes #962

commit 19a8407f741848db0dc86e587ea9739b17888768
Author: Gus Katsiapis <katsia...@katsiapis-linux.mtv.corp.google.com>
Date:   2016-09-20T00:

[GitHub] incubator-beam pull request #1443: [BEAM-1055] Display Data keys on Python a...

2016-11-28 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1443

[BEAM-1055] Display Data keys on Python are inconsistent

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit e620b07afd10439c29401726161cb7139185ec38
Author: Pablo <pabl...@google.com>
Date:   2016-11-28T21:05:47Z

Standardizing display data keys




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1420: [BEAM-147] Adding Metrics API to Python S...

2016-11-23 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1420

[BEAM-147] Adding Metrics API to Python SDK

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit d5ced340e83d850d2dfbe1cfed2b28100bf45d80
Author: Pablo <pabl...@google.com>
Date:   2016-11-22T22:31:26Z

Progress on Metrics API

commit 5aa49dfe829debc38c2fcd48178bb93512dff026
Author: Pablo <pabl...@google.com>
Date:   2016-11-23T18:49:48Z

Refactoring to fix circle dependencies

commit 74977da4c93944c8b673956c42d78fbca1e847d8
Author: Pablo <pabl...@google.com>
Date:   2016-11-23T19:48:51Z

Adding basic documentation. Implementing MetricsFilter

commit b9a4fee7922d38a6413ed2bb1e55b03e853742eb
Author: Pablo <pabl...@google.com>
Date:   2016-11-23T22:20:11Z

Fixing typo. Adding documentation

commit 4b19e7d780a9c11260ad05d18aa8dea1f53472b3
Author: Pablo <pabl...@google.com>
Date:   2016-11-23T22:47:56Z

Fixing typo-s




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1420: [BEAM-147] Adding Metrics API to Python S...

2016-11-23 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1420


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1419: [BEAM-147] Adding Metrics API to Python S...

2016-11-22 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1419

[BEAM-147] Adding Metrics API to Python SDK

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit 937cf69e958d4a82fb274f311de248930298db69
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-22T21:32:33Z

Fix multi-input named PTransforms.

Now delegate the __ror__ logic entirely for the naming wrapper.

commit 031c4cce0b9eae0d50a49f43ffeced1edbfd2f8f
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-22T21:34:58Z

Move names out of transform constructors.

sed -i -r 's/[|] (\S+)[(](["'"'"'][^"'"'"']+.)(, +|([)]))/| \2 >> \1(\4/g'

Small number of tests will need to be fixed by hand.

commit 7c186ce4ce18c98cf3979b6f0f07c72a47b42ec9
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-22T22:55:26Z

Fix error messages for externally named PTransforms.

commit 362f2e9e4398662d55a2a4e6399f43155c58ed24
Author: Ahmet Altay <al...@google.com>
Date:   2016-07-22T23:03:08Z

fix pipeline test

commit 2ff5630be06034e25d60e105bbe1a718ae06b4d6
Author: Chamikara Jayalath <chamik...@google.com>
Date:   2016-07-22T23:04:29Z

Fixes examples

commit 01830510bded3c22ddd96937f5d83547702a4385
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-22T23:05:23Z

Cleanup and fix combiners_test.

commit 2a59a121441a003bad949ae6a23d58a9cf2b3059
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-22T23:24:48Z

Fix label-sensitive test.

commit b15d35ca6e585e75153e05d96403336889cc6894
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-23T01:35:22Z

Lint fixes.

commit c5b5b14d35fc7f6b0a576f0fca19b730015e1282
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-23T08:07:44Z

fixup: failing tests expecting name

commit e3c078fe28553b7e7317316b6df51b4c570573ba
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-23T08:32:23Z

Final cleanup pass.

commit 38d9dea2e62af280e4b9c258cedee70d6bcaa8ca
Author: Robert Bradshaw <rober...@gmail.com>
Date:   2016-07-23T23:43:47Z

Closes #718

commit ac3249fe51e9674149ae9cccd48c7290aa8bf34d
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-23T00:33:57Z

Make DoFnRunner a Receiver.

commit d20cf64287fdd22b1efa1a3afe346a0a1d5cf924
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-23T07:37:42Z

Receiver and LoggingContext adapters.

This allows for cdef calls iff the right type is passed in.

commit de35f28294dfb4f47b429635eff843b38279e240
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-23T07:54:11Z

Allow passing logging context directly.

This is better than passing a logger module with a specific class.

commit 153916fe9f641f5fe5e8d473caef67f449ee6bca
Author: Robert Bradshaw <rober...@google.com>
Date:   2016-07-25T19:02:02Z

Closes #721

commit 8a7bc71dee1e3f9f17c29e5e558870a6bc5f4880
Author: Charles Chen <c...@google.com>
Date:   2016-07-26T01:22:08Z

Fix SDK name and version sent to the Cloud Dataflow service

commit 76f3864b61ad9e2ac9a2e81e7a0d5993db5fde6c
Author: Charles Chen <c...@google.com>
Date:   2016-07-26T01:36:45Z

Update docstring.

commit 26ff6579531d598e91b220fa21ec9bd7f9220f78
Author: Robert Bradshaw <rober...@gmail.com>
Date:   2016-07-26T04:11:31Z

Closes #730

commit 2f4054ba37da1c1100f45a572d96e7a6e2e60152
Author: Charles Chen <c...@google.com>
Date:   2016-07-25T18:44:22Z

Make TextFileReader observable

This allows future implementation of size tracking for elements in side 
input sources.

commit 53ab635c75e4b94b3930a601911a717ddc499efe
Author: Robert Bradshaw <rober...@gmail.com>
Date:   2016-07-28T18:05:56Z

Closes #726

commit adb3ed93053c83b4e28e7baa879e9aee82f02785
Author: Robert

[GitHub] incubator-beam pull request #1419: [BEAM-147] Adding Metrics API to Python S...

2016-11-22 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1419


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1395: Fixing error with PipelineOptions Display...

2016-11-21 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1395

Fixing error with PipelineOptions DisplayData of lists

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---



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

$ git pull https://github.com/pabloem/incubator-beam dd-options-list

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

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


commit a361cf5c0e72babbc7fae58b6401b6fdce555181
Author: Pablo <pabl...@google.com>
Date:   2016-11-21T19:15:37Z

Fixing error with PipelineOptions DisplayData of lists

commit bac792a759b3269daefe4b3d47d1e4de46e5c4fe
Author: Pablo <pabl...@google.com>
Date:   2016-11-21T21:52:43Z

Improving documentation




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1395: Fixing error with PipelineOptions Display...

2016-11-21 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1395

Fixing error with PipelineOptions DisplayData of lists

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---



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

$ git pull https://github.com/pabloem/incubator-beam dd-options-list

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

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


commit 942d0c1cb775876b1a6bd91959a60aa9ae576aa7
Author: Pablo <pabl...@google.com>
Date:   2016-11-21T19:15:37Z

Fixing error with PipelineOptions DisplayData of lists




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1264: [BEAM-884] Adding display data to Pipelin...

2016-11-16 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1264


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1212: [BEAM-850] Adding display data to sink/so...

2016-11-09 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1212


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1273: Accepting integers of type `long` that fi...

2016-11-03 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1273


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1273: Accepting integers of type `long` that fi...

2016-11-03 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1273

Accepting integers of type `long` that fit in 64 bits.

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

$ git pull https://github.com/pabloem/incubator-beam windows-timestamp-break

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

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


commit 240625a9e4c97154e410c0025d0a5705ff8f54ad
Author: Pablo <pabl...@google.com>
Date:   2016-11-03T18:41:53Z

Checking for integer types in json conversion

commit cdb7c3f8170f6c67876092856f939b6267270110
Author: Pablo <pabl...@google.com>
Date:   2016-11-03T18:45:23Z

Fixing lint




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1200: [BEAM-824] Fixing error message when sdk_...

2016-11-03 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1200


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1264: [BEAM-884] Adding display data to Pipelin...

2016-11-02 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1264

[BEAM-884] Adding display data to PipelineOptions and others.

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

$ git pull https://github.com/pabloem/incubator-beam display-data-options

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

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


commit ae9e26cdd1e61a08326d08397b91b246593949ee
Author: Pablo <pabl...@google.com>
Date:   2016-10-27T19:07:02Z

Adding display data to sink/sources PubSub, BigQuery, NativeFile, 
FileBasedSource; Write, Read, ParDo transforms; DoFn.

commit 247105f575d353118e53a1d6bca9aaa1453ac9e3
Author: Pablo <pabl...@google.com>
Date:   2016-10-27T19:43:08Z

Adding hamcrest dependency

commit 478679e4902289fe5af5504b4387a046c7946e61
Author: Pablo <pabl...@google.com>
Date:   2016-10-31T20:44:28Z

Addressing comments by swegner

commit de01f1416613b1b6588efc7ae92c6fdf983c201e
Author: Pablo <pabl...@google.com>
Date:   2016-10-31T23:22:51Z

Adding ItemMatcher for display data unittests

commit 1c5fbb08404cb00c3ab9444955414f2f3ee8f863
Author: Pablo <pabl...@google.com>
Date:   2016-11-01T16:38:21Z

Adding pydoc

commit 922d16b04d7ad5fbe0ed91124346ba65983ec54c
Author: Pablo <pabl...@google.com>
Date:   2016-11-01T18:57:17Z

Addressed comments by swegner

commit c5b90c0c7896a9495c0a18e179e83c47c4fe146c
Author: Pablo <pabl...@google.com>
Date:   2016-11-01T20:04:23Z

Renamed DisplayDataItemMatcher

commit b13be5c1d8bcecbe74fc1753de597b64116f6afd
Author: Pablo <pabl...@google.com>
Date:   2016-11-01T21:09:13Z

Adding test_require in setup.py

commit ac89d1ee7e94dcf317a1c46227d79a334b3bf0dc
Author: Pablo <pabl...@google.com>
Date:   2016-11-01T23:34:11Z

Addressing ccy's comments

commit 6167b8dd63fc4686097613ec608c0d06d7061186
Author: Pablo <pabl...@google.com>
Date:   2016-11-01T23:44:10Z

Fixing spelling of dummy

commit f6e9a5bbcc29728ee1983e98783f1a1b18c05379
Author: Pablo <pabl...@google.com>
Date:   2016-11-02T16:59:17Z

Adding extra check to display data unit test

commit 58877ab5b23999c4729a8054a2d629d1e4ffc6b0
Author: Pablo <pabl...@google.com>
Date:   2016-11-02T20:06:15Z

Starting to add display data to extra 
sinks/sources/transforms/PipelineOptions




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1112: [BEAM-722] Display data infrastructure

2016-10-27 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1112


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1200: [BEAM-824] Fixing error message when sdk_...

2016-10-26 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1200

[BEAM-824] Fixing error message when sdk_location not provided

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---

r: @aaltay can you take a look?
I tried adding the error strings to `pkg_resources.DistributionNotFound`'s 
`.args` tuple, but it seems to have special formatting for them so I couldn't 
add extra information on the error message. Let me know how to change the error 
message.

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

$ git pull https://github.com/pabloem/incubator-beam 
sdk-location-err-message

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

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






---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1112: [BEAM-722] Display data infrastructure

2016-10-14 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1112

[BEAM-722] Display data infrastructure

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [x] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [x] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

$ git pull https://github.com/pabloem/incubator-beam 
display-data-infrastructure

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

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


commit f3c7ebd24ecfd0b46aa4b2d6c906c4c1331fd13a
Author: Pablo <pabl...@google.com>
Date:   2016-10-14T18:44:47Z

Laying down infrastructure for static display data




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1035: [BEAM] Making sure that GcsBufferedReader...

2016-10-10 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1035


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1035: [BEAM] Making sure that GcsBufferedReader...

2016-10-10 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1035


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1035: [BEAM] Making sure that GcsBufferedReader...

2016-10-10 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1035

[BEAM] Making sure that GcsBufferedReader implements the iterator protocol

The change adds only the `__iter__` and `next` functions to the 
`GcsBufferedReader` class, so that it implements the iterator protocol in 
Python.


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

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

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

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






---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-10 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1068


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1068


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1068


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1068

[BEAM-721] Testing tox intermittent failures on Mac

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ x ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ x ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ x ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit 20ffef06dec7a18d52aa84c74c2a91727af10c9d
Author: Pablo <pabl...@google.com>
Date:   2016-10-07T22:19:48Z

Fixing the issue where tox would be installed in some Mac images in
Travis CI, but not others.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1068


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1068

[BEAM-721] Testing tox intermittent failures on Mac

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ x ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ x ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ x ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit 20ffef06dec7a18d52aa84c74c2a91727af10c9d
Author: Pablo <pabl...@google.com>
Date:   2016-10-07T22:19:48Z

Fixing the issue where tox would be installed in some Mac images in
Travis CI, but not others.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1068

[BEAM-721] Testing tox intermittent failures on Mac

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ x ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ x ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ x ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit 20ffef06dec7a18d52aa84c74c2a91727af10c9d
Author: Pablo <pabl...@google.com>
Date:   2016-10-07T22:19:48Z

Fixing the issue where tox would be installed in some Mac images in
Travis CI, but not others.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
GitHub user pabloem reopened a pull request:

https://github.com/apache/incubator-beam/pull/1068

[BEAM-721] Testing tox intermittent failures on Mac

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ x ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ x ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ x ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit 20ffef06dec7a18d52aa84c74c2a91727af10c9d
Author: Pablo <pabl...@google.com>
Date:   2016-10-07T22:19:48Z

Fixing the issue where tox would be installed in some Mac images in
Travis CI, but not others.




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1068


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1068: [BEAM-721] Testing tox intermittent failu...

2016-10-07 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1068

[BEAM-721] Testing tox intermittent failures on Mac

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ x ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ x ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ x ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---


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

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

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

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


commit a76b10df839f7281388e1d36a2d33f71a4527ff8
Author: Pablo <pabl...@google.com>
Date:   2016-10-07T22:19:48Z

Trying to fix tox on Mac




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1062: [BEAM-721] Fixing tox for python

2016-10-06 Thread pabloem
Github user pabloem closed the pull request at:

https://github.com/apache/incubator-beam/pull/1062


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-beam pull request #1062: [BEAM-721] Fixing tox for python

2016-10-06 Thread pabloem
GitHub user pabloem opened a pull request:

https://github.com/apache/incubator-beam/pull/1062

[BEAM-721] Fixing tox for python

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

 - [ x ] Make sure the PR title is formatted like:
   `[BEAM-] Description of pull request`
 - [ ] Make sure tests pass via `mvn clean verify`. (Even better, enable
   Travis-CI on your fork and ensure the whole test matrix passes).
 - [ ] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [ ] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.txt).

---
Just testing for now.

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

$ git pull https://github.com/pabloem/incubator-beam test-tox-python

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

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


commit b9fcd5f2e56d8e7eb13a8937f294b1eeb2b705b9
Author: polecito...@gmail.com <polecito...@gmail.com>
Date:   2016-09-29T22:41:25Z

Making sure that GcsBufferedReader implements the iterator protocol

commit 2d93943fad5e8179909e21dac739e3a5ce9eab55
Author: polecito...@gmail.com <polecito...@gmail.com>
Date:   2016-10-04T22:40:43Z

Adding raise StopIteration to GcsBufferedReader.next

commit 30c66d8919cb038fa01f7d46ed66294d51201a8f
Author: polecito...@gmail.com <polecito...@gmail.com>
Date:   2016-10-05T00:03:14Z

Adding unit tests, and __next__ for Python 3

commit 475f7e0c475c8939cd685e2761b1bd02486ca884
Author: polecito...@gmail.com <polecito...@gmail.com>
Date:   2016-10-05T00:22:19Z

Fixing some lint issues in unit test

commit 387f824d15b6d9caaed8d01633ca162aadef9797
Author: polecito...@gmail.com <polecito...@gmail.com>
Date:   2016-10-06T00:47:52Z

Improving test readability.

commit 51b4e775af86891f41efdfb0136317c9724e40b7
Author: Pablo <pabl...@google.com>
Date:   2016-10-06T20:44:34Z

Trying to unify the TOX_HOME variable




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---