[GitHub] beam pull request #2809: BEAM-1773 Consider allowing Source#validate() to th...

2017-05-25 Thread tedyu
Github user tedyu closed the pull request at: https://github.com/apache/beam/pull/2809 --- 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

[GitHub] beam pull request #2809: BEAM-1773 Consider allowing Source#validate() to th...

2017-05-01 Thread tedyu
GitHub user tedyu opened a pull request: https://github.com/apache/beam/pull/2809 BEAM-1773 Consider allowing Source#validate() to throw exception See if this approach looks good. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] beam pull request #2332: BEAM-1760 Potential null dereference in HDFSFileSin...

2017-04-03 Thread tedyu
Github user tedyu closed the pull request at: https://github.com/apache/beam/pull/2332 --- 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

[GitHub] beam pull request #2332: BEAM-1760 Potential null dereference in HDFSFileSin...

2017-03-26 Thread tedyu
GitHub user tedyu opened a pull request: https://github.com/apache/beam/pull/2332 BEAM-1760 Potential null dereference in HDFSFileSink#doFinalize Check whether s.getPath().getParent() is null. If it is null, break out of the loop. You can merge this pull request into a Git

[GitHub] beam pull request #2227: BEAM-1568 neffective null check in IsmFormat#struct...

2017-03-11 Thread tedyu
GitHub user tedyu opened a pull request: https://github.com/apache/beam/pull/2227 BEAM-1568 neffective null check in IsmFormat#structuralValue Add check for record at the beginning of the method. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] beam pull request #2121: BEAM-1567 hashStream should be closed in PackageUti...

2017-02-27 Thread tedyu
GitHub user tedyu opened a pull request: https://github.com/apache/beam/pull/2121 BEAM-1567 hashStream should be closed in PackageUtil#createPackageAttributes() Here is related code: ``` OutputStream hashStream = Funnels.asOutputStream(hasher); ``` hashStream