[GitHub] beam pull request #4057: [BEAM-3035] Extract ReifyTimestampsAndWindows

2017-11-10 Thread wtanaka
Github user wtanaka closed the pull request at:

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


---


[GitHub] beam pull request #4057: [BEAM-3035] Extract ReifyTimestampsAndWindows

2017-11-10 Thread wtanaka
GitHub user wtanaka reopened a pull request:

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

[BEAM-3035] Extract ReifyTimestampsAndWindows

from GatherAllPanes

---

 - [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.
 - [ ] Write a pull request description that is detailed enough to 
understand what the pull request does, how, and why.
 - [ ] Run `mvn clean verify` to make sure basic checks pass. A more 
thorough check will be performed on your pull request automatically.
 - [x] If this contribution is large, please file an Apache [Individual 
Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).


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

$ git pull https://github.com/wtanaka/beam reify

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

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


commit c4062cc74e6efde4f86318dd23302bb0fc296e6d
Author: wtanaka.com 
Date:   2017-10-07T18:45:45Z

[BEAM-3035] Extract ReifyTimestampsAndWindows

from GatherAllPanes




---


[GitHub] beam pull request #4063: Remove redundant public modifier on inner interface...

2017-10-30 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Remove redundant public modifier on inner interfaces



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

$ git pull https://github.com/wtanaka/beam redundantpublic

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

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


commit a31759f1bca66fb1dc24ca67aaaf963992c465fc
Author: wtanaka.com 
Date:   2017-10-07T18:57:02Z

Remove redundant public modifier on inner interfaces




---


[GitHub] beam pull request #4057: [BEAM-3035] Extract ReifyTimestampsAndWindows

2017-10-29 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

[BEAM-3035] Extract ReifyTimestampsAndWindows

from GatherAllPanes

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

$ git pull https://github.com/wtanaka/beam reify

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

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


commit e9b5cf823813ea01ff161179619ec196d463cdf2
Author: wtanaka.com 
Date:   2017-10-07T18:45:45Z

[BEAM-3035] Extract ReifyTimestampsAndWindows

from GatherAllPanes




---


[GitHub] beam pull request #3959: Remove semicolon

2017-10-07 Thread wtanaka
Github user wtanaka closed the pull request at:

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


---


[GitHub] beam pull request #3959: Remove semicolon

2017-10-07 Thread wtanaka
GitHub user wtanaka reopened a pull request:

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

Remove semicolon



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

$ git pull https://github.com/wtanaka/beam removesemi

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

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


commit a97d7d819e1bc360559b44dab0a0f6bbf9a1c890
Author: wtanaka.com 
Date:   2017-10-07T19:05:58Z

Remove semicolon




---


[GitHub] beam pull request #3960: Utilize some java 5 features

2017-10-07 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Utilize some java 5 features

* Remove some unnecessary boxing/unboxing
* Replace for loops with foreach

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

$ git pull https://github.com/wtanaka/beam java5

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

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


commit e8bc2bdc0577e64c59f80ed1ae42ea87dcf6b3e5
Author: wtanaka.com 
Date:   2017-10-07T19:22:45Z

Utilize some java 5 features

* Remove some unnecessary boxing/unboxing
* Replace for loops with foreach




---


[GitHub] beam pull request #3959: Remove semicolon

2017-10-07 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Remove semicolon



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

$ git pull https://github.com/wtanaka/beam removesemi

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

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


commit a97d7d819e1bc360559b44dab0a0f6bbf9a1c890
Author: wtanaka.com 
Date:   2017-10-07T19:05:58Z

Remove semicolon




---


[GitHub] beam pull request #3773: Small tweak to View.asList javadoc

2017-08-27 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Small tweak to View.asList javadoc

This may help clarify


https://lists.apache.org/thread.html/cd9bd1ae4b6945cd78e04b3baa7628bd43071c443a752acbb83d388d@%3Cdev.beam.apache.org%3E

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

$ git pull https://github.com/wtanaka/beam docs

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

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


commit d27d056464a0deee3080df6351506fedf115161c
Author: wtanaka.com 
Date:   2017-08-27T20:23:21Z

Small tweak to View.asList javadoc

This may help clarify


https://lists.apache.org/thread.html/cd9bd1ae4b6945cd78e04b3baa7628bd43071c443a752acbb83d388d@%3Cdev.beam.apache.org%3E




---
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] beam pull request #3231: [BEAM-2363] Function composition for SerializableFn

2017-05-25 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

[BEAM-2363] Function composition for SerializableFn

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`.
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [x] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.pdf).

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`.
 - [ ] 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.pdf).

---


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

$ git pull https://github.com/wtanaka/beam dev

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

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


commit 11bf95483a112432681f389624dbac78c9f20165
Author: wtanaka.com 
Date:   2017-05-26T05:31:38Z

[BEAM-2363] Function composition for SerializableFn

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`.
 - [x] Replace `` in the title with the actual Jira issue
   number, if there is one.
 - [x] If this contribution is large, please file an Apache
   [Individual Contributor License 
Agreement](https://www.apache.org/licenses/icla.pdf).




---
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] beam pull request #2654: IntelliJ gives "ambiguous method call" error

2017-05-20 Thread wtanaka
Github user wtanaka closed the pull request at:

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


---
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] beam pull request #3139: Clean up exception signatures

2017-05-14 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Clean up exception signatures

Strengthen some "throws Exception" declarations

CoderException extends IOException so it's not necessary to declare
both in "throws"


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

---

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

$ git pull https://github.com/wtanaka/beam BEAM-2293

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

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


commit 5dec09f884b5a085bd528f6881d8334930bc0a89
Author: wtanaka.com 
Date:   2017-05-14T18:55:27Z

[BEAM-2293] Rename Top.{Largest,Smallest=>Natural,Reversed}

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`.
 - [ ] 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.pdf).




---
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] beam pull request #2789: Clean up exception signatures

2017-05-13 Thread wtanaka
Github user wtanaka closed the pull request at:

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


---
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] beam pull request #2789: Clean up exception signatures

2017-04-29 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Clean up exception signatures

Strengthen some "throws Exception" declarations

CoderException extends IOException so it's not necessary to declare
both in "throws"

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

$ git pull https://github.com/wtanaka/beam exceptionsigs

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

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


commit 90432eb214f975b3de512e0c7ff536618d9af003
Author: wtanaka.com 
Date:   2017-04-30T04:45:42Z

Remove useless return statement

commit e60d29974adc59912e381df0ef5abb027fb06c87
Author: wtanaka.com 
Date:   2017-04-30T05:20:59Z

Clean up exception signatures

Strengthen some "throws Exception" declarations

CoderException extends IOException so it's not necessary to declare
both in "throws"




---
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] beam pull request #2788: Remove useless return statement

2017-04-29 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Remove useless return statement

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

---

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

$ git pull https://github.com/wtanaka/beam uselessreturn

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

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


commit 90432eb214f975b3de512e0c7ff536618d9af003
Author: wtanaka.com 
Date:   2017-04-30T04:45:42Z

Remove useless return statement




---
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] beam pull request #2787: Remove useless continue statements

2017-04-29 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Remove useless continue statements



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

$ git pull https://github.com/wtanaka/beam unnecessarycontinue

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

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


commit 5b3a7d4a17afa687eedfbc205c682a2a1279624d
Author: wtanaka.com 
Date:   2017-04-30T04:44:06Z

Remove useless continue statements




---
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] beam pull request #2786: Remove redundant private on enum constructors

2017-04-29 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Remove redundant private on enum constructors

https://docs.oracle.com/javase/specs/jls/se8/html/jls-8.html#jls-8.9.2

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

---

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

$ git pull https://github.com/wtanaka/beam enumconstruct

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

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


commit a11ff41c4df4fcb70c17bae2b8af5ae36b99cd94
Author: wtanaka.com 
Date:   2017-04-30T04:27:43Z

Remove redundant private on enum constructors

https://docs.oracle.com/javase/specs/jls/se8/html/jls-8.html#jls-8.9.2




---
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] beam pull request #2699: Fix small typo in comment in CoderRegistry

2017-04-26 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Fix small typo in comment in CoderRegistry



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

$ git pull https://github.com/wtanaka/beam typo

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

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


commit e3f6cf9422d95646242c9a897205a4bb8b0546b2
Author: wtanaka.com 
Date:   2017-04-26T08:15:57Z

Fix small typo in comment in CoderRegistry




---
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] beam pull request #2655: Rename openWindows => windowsThatAreOpen

2017-04-22 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Rename openWindows => windowsThatAreOpen

to make its name have parallel structure with windowsThatShouldFire

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

$ git pull https://github.com/wtanaka/beam rename

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

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


commit 59a580735384fd72374f263588b41819e9a415e1
Author: wtanaka.com 
Date:   2017-04-23T01:48:32Z

Rename openWindows => windowsThatAreOpen

to make its name have parallel structure with windowsThatShouldFire




---
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] beam pull request #2654: IntelliJ gives "ambiguous method call" error

2017-04-22 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

IntelliJ gives "ambiguous method call" error

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

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

$ git pull https://github.com/wtanaka/beam intellij

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

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


commit 9745c409af6495eb360d8e481817aeb1c042f1c6
Author: wtanaka.com 
Date:   2017-04-22T22:47:38Z

IntelliJ gives "ambiguous method call" error




---
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] beam pull request #2653: Add comments to Trigger.java

2017-04-22 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Add comments to Trigger.java



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

$ git pull https://github.com/wtanaka/beam comments

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

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


commit 7193c0460fe741308fe16745a5090ac5dbf07c2a
Author: wtanaka.com 
Date:   2017-04-22T22:14:35Z

Add comments to Trigger.java




---
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] beam pull request #2652: Remove unused private fields

2017-04-22 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Remove unused private fields



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

$ git pull https://github.com/wtanaka/beam dev

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

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


commit d45df9f6265a443dfe85fdcb713cf27e751199f3
Author: wtanaka.com 
Date:   2017-04-22T21:44:17Z

Remove unused private fields




---
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] beam pull request #2650: Remove unnecessary semicolons

2017-04-22 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Remove unnecessary semicolons

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

---

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

$ git pull https://github.com/wtanaka/beam semicolons

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

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


commit a1e99b16bdca8a1832037ac6b49a20541c3e0f3c
Author: wtanaka.com 
Date:   2017-04-22T18:18:58Z

Remove unnecessary semicolons




---
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] beam pull request #2577: [BEAM-1992] Move Count.PerElement javadoc to a publ...

2017-04-18 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

[BEAM-1992] Move Count.PerElement javadoc to a public place

- [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.pdf).

---

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

$ git pull https://github.com/wtanaka/beam BEAM-1992

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

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


commit 0413593e6b6cb54594f684c21173a6b5ad0aedd7
Author: wtanaka.com 
Date:   2017-04-18T07:57:10Z

[BEAM-1992] Move Count.PerElement javadoc to a public place

 - [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.pdf).

---




---
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] beam pull request #2565: [BEAM-1991] Sum.SumDoubleFn => Sum.ofDoubles

2017-04-17 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

[BEAM-1991] Sum.SumDoubleFn => Sum.ofDoubles

This is left over from commit 78a360eac35507d9a558fc6117bb56b67b8a884e

 - [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.pdf).

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

$ git pull https://github.com/wtanaka/beam BEAM-1991

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

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


commit b11dac9193d343873d050422c4a8d92ad626be1a
Author: wtanaka.com 
Date:   2017-04-18T02:45:45Z

[BEAM-1991] Sum.SumDoubleFn => Sum.ofDoubles

This is left over from commit 78a360eac35507d9a558fc6117bb56b67b8a884e

 - [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.pdf).




---
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] beam pull request #2564: [BEAM-1990] Make Window.Assign default visibility

2017-04-17 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

[BEAM-1990] Make Window.Assign default visibility

- [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.pdf).

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

$ git pull https://github.com/wtanaka/beam window-assign

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

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


commit 8591dc9f4367aa7e11221bf24b85891bfe886979
Author: wtanaka.com 
Date:   2017-04-18T01:31:13Z

[BEAM-1990] Make Window.Assign default visibility

 - [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.pdf).




---
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] beam pull request #2406: Add to javadoc of getCheckpointMark

2017-04-03 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Add to javadoc of getCheckpointMark

Based on looking through various implementations of this method

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

$ git pull https://github.com/wtanaka/beam checkpoint

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

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


commit 54832ddadd6e40a027f6490124c489a071fdcdf3
Author: wtanaka.com 
Date:   2017-04-03T09:02:00Z

Add to javadoc of getCheckpointMark

Based on looking through various implementations of this method




---
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] beam pull request #2405: Add javadoc to getEstimatedSizeBytes

2017-04-02 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Add javadoc to getEstimatedSizeBytes

based on code comment in BoundedReadFromUnboundedSource.java

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

$ git pull https://github.com/wtanaka/beam get-est

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

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


commit d2977096d15047a0fdd2bbf7fd7d9149b635f66e
Author: wtanaka.com 
Date:   2017-04-03T06:02:02Z

Add javadoc to getEstimatedSizeBytes

based on code comment in BoundedReadFromUnboundedSource.java




---
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] beam pull request #2402: Add example to BoundedCountingInput javadoc

2017-04-01 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Add example to BoundedCountingInput javadoc



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

$ git pull https://github.com/wtanaka/beam example

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

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


commit 3345c2bae09a2f3653ffb81e2771bac97e2ad5c2
Author: wtanaka.com 
Date:   2017-04-02T02:44:14Z

Add example to BoundedCountingInput javadoc




---
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] beam pull request #2401: New copy for error when Create called w/zero items

2017-04-01 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

New copy for error when Create called w/zero items



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

$ git pull https://github.com/wtanaka/beam master

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

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


commit 05eb175506c8ffc2dc6f16c3bf38566c884beee1
Author: wtanaka.com 
Date:   2017-04-01T21:29:31Z

New copy for error when Create called w/zero items




---
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] beam pull request #2326: [BEAM-1809] Make TravisCI builds less broken

2017-03-27 Thread wtanaka
Github user wtanaka closed the pull request at:

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


---
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] beam pull request #2329: Fix type signature typo in Javadoc

2017-03-26 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Fix type signature typo in Javadoc



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

$ git pull https://github.com/wtanaka/beam signaturetypo

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

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


commit fe33b3b0d7483dcb294ab5d14c6e36029d39da12
Author: wtanaka.com 
Date:   2017-03-26T11:07:54Z

Fix type signature typo in Javadoc




---
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] beam pull request #2326: Make TravisCI builds less broken

2017-03-25 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Make TravisCI builds less broken

Before:


![before](https://cloud.githubusercontent.com/assets/35872/24320714/c1bd2efc-10de-11e7-9b7b-4f4786767066.png)

After:

![screenshot from 2017-03-24 
22-10-02](https://cloud.githubusercontent.com/assets/35872/24320718/cf7e8248-10de-11e7-9254-cb4d954f394a.png)


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

$ git pull https://github.com/wtanaka/beam checkstyle

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

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


commit da2540fe9f9ddc59f550f2294d8b0b99f247c60a
Author: wtanaka.com 
Date:   2017-03-25T04:15:47Z

Fix org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check

[ERROR] Failed to execute goal
org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (default) on
project beam-sdks-parent: Failed during checkstyle execution: Unable to find
suppressions file at location: beam/suppressions.xml: Could not find 
resource
'beam/suppressions.xml'. -> [Help 1]

commit d713f85726b019b526fd306377e467330379c1e4
Author: wtanaka.com 
Date:   2017-03-25T07:13:39Z

Fix org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check

[ERROR] Failed to execute goal
org.apache.maven.plugins:maven-checkstyle-plugin:2.17:check (default) on
project beam-sdks-parent: Failed during checkstyle execution: Unable to find
suppressions file at location: beam/suppressions.xml: Could not find 
resource
'beam/suppressions.xml'. -> [Help 1]




---
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] beam pull request #2312: Fix typo isPastEndOfWindow => pastEndOfWindow

2017-03-24 Thread wtanaka
GitHub user wtanaka opened a pull request:

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

Fix typo isPastEndOfWindow => pastEndOfWindow



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

$ git pull https://github.com/wtanaka/beam docs

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

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


commit cdcca38e56da77d6bc20c34e5b0716f213ba851a
Author: Wesley Tanaka 
Date:   2017-03-24T09:02:20Z

Fix typo isPastEndOfWindow => pastEndOfWindow




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