[jira] [Commented] (BEAM-1569) HDFSFileSource: Unable to read from filePattern with spaces in path

2017-02-27 Thread JIRA

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

Jean-Baptiste Onofré commented on BEAM-1569:


Yes, I agree. A PR would be helpful. Ready to review !

> HDFSFileSource: Unable to read from filePattern with spaces in path
> ---
>
> Key: BEAM-1569
> URL: https://issues.apache.org/jira/browse/BEAM-1569
> Project: Beam
>  Issue Type: Bug
>  Components: sdk-java-core
>Reporter: Michael Luckey
>Assignee: Michael Luckey
>
> After the merge of the changes introduced with 
> https://issues.apache.org/jira/browse/BEAM-1497 we are unable to read from 
> files containing spaces in path. We encounter following stack trace
> {noformat}
> java.lang.reflect.UndeclaredThrowableException
>   at 
> org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1713)
>   at 
> org.apache.beam.sdk.io.hdfs.HDFSFileSource.validate(HDFSFileSource.java:337)
>   at 
> org.apache.beam.sdk.io.hdfs.HDFSFileSource.createReader(HDFSFileSource.java:329)
>   at 
> org.apache.beam.sdk.testing.SourceTestUtils.readFromSource(SourceTestUtils.java:138)
> Caused by: java.net.URISyntaxException: Illegal character in path at index 
> 77: 
> /var/folders/1t/s9pcmfj50nxbt68h3_2z_5wcgn/T/junit6887354597440386901/tmp 
> data.seq
>   at java.net.URI$Parser.fail(URI.java:2848)
>   at java.net.URI$Parser.checkChars(URI.java:3021)
>   at java.net.URI$Parser.parseHierarchical(URI.java:3105)
>   at java.net.URI$Parser.parse(URI.java:3063)
>   at java.net.URI.(URI.java:588)
>   at 
> org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:340)
>   at 
> org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:337)
>   at java.security.AccessController.doPrivileged(Native Method)
>   at javax.security.auth.Subject.doAs(Subject.java:422)
>   at 
> org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1698)
>   ... 40 more
> {noformat}
> This can be reproduced for instance by 
> {noformat}
>   // shameless copy of existing test case
>   @Test
>   public void testFullyReadSingleFileWithSpaces() throws Exception {
> PipelineOptions options = PipelineOptionsFactory.create();
> List> expectedResults = createRandomRecords(3, 10, 
> 0);
> File file = createFileWithData("tmp data.seq", expectedResults);
> HDFSFileSource, IntWritable, Text> source =
> HDFSFileSource.from(
> file.toString(), SequenceFileInputFormat.class, 
> IntWritable.class, Text.class);
> assertEquals(file.length(), source.getEstimatedSizeBytes(null));
> assertThat(expectedResults, containsInAnyOrder(readFromSource(source, 
> options).toArray()));
>   }
> {noformat}
> Changing the implementation slightly to
> {noformat}
> diff --git 
> a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
>  
> b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
> index 2a731fb..df72643 100644
> --- 
> a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
> +++ 
> b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
> @@ -30,7 +30,6 @@ import java.io.ObjectInput;
>  import java.io.ObjectOutput;
>  import java.lang.reflect.InvocationTargetException;
>  import java.lang.reflect.Method;
> -import java.net.URI;
>  import java.security.PrivilegedExceptionAction;
>  import java.util.List;
>  import java.util.ListIterator;
> @@ -337,9 +336,10 @@ public abstract class HDFSFileSource extends 
> BoundedSource {
>  UGIHelper.getBestUGI(username()).doAs(new 
> PrivilegedExceptionAction() {
>@Override
>public Void run() throws Exception {
> -FileSystem fs = FileSystem.get(new URI(filepattern()),
> +final Path pathPattern = new Path(filepattern());
> +FileSystem fs = FileSystem.get(pathPattern.toUri(),
>  
> SerializableConfiguration.newConfiguration(serializableConfiguration()));
> -FileStatus[] fileStatuses = fs.globStatus(new 
> Path(filepattern()));
> +FileStatus[] fileStatuses = fs.globStatus(pathPattern);
>  checkState(
>  fileStatuses != null && fileStatuses.length > 0,
>  "Unable to find any files matching %s", filepattern());
> {noformat}
> seems to be fixing the issue for us.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (BEAM-1569) HDFSFileSource: Unable to read from filePattern with spaces in path

2017-02-27 Thread JIRA

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

Jean-Baptiste Onofré commented on BEAM-1569:


By the way, the refactoring of the HDFS IO broke bunch of things in our code 
too.

> HDFSFileSource: Unable to read from filePattern with spaces in path
> ---
>
> Key: BEAM-1569
> URL: https://issues.apache.org/jira/browse/BEAM-1569
> Project: Beam
>  Issue Type: Bug
>  Components: sdk-java-core
>Reporter: Michael Luckey
>Assignee: Michael Luckey
>
> After the merge of the changes introduced with 
> https://issues.apache.org/jira/browse/BEAM-1497 we are unable to read from 
> files containing spaces in path. We encounter following stack trace
> {noformat}
> java.lang.reflect.UndeclaredThrowableException
>   at 
> org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1713)
>   at 
> org.apache.beam.sdk.io.hdfs.HDFSFileSource.validate(HDFSFileSource.java:337)
>   at 
> org.apache.beam.sdk.io.hdfs.HDFSFileSource.createReader(HDFSFileSource.java:329)
>   at 
> org.apache.beam.sdk.testing.SourceTestUtils.readFromSource(SourceTestUtils.java:138)
> Caused by: java.net.URISyntaxException: Illegal character in path at index 
> 77: 
> /var/folders/1t/s9pcmfj50nxbt68h3_2z_5wcgn/T/junit6887354597440386901/tmp 
> data.seq
>   at java.net.URI$Parser.fail(URI.java:2848)
>   at java.net.URI$Parser.checkChars(URI.java:3021)
>   at java.net.URI$Parser.parseHierarchical(URI.java:3105)
>   at java.net.URI$Parser.parse(URI.java:3063)
>   at java.net.URI.(URI.java:588)
>   at 
> org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:340)
>   at 
> org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:337)
>   at java.security.AccessController.doPrivileged(Native Method)
>   at javax.security.auth.Subject.doAs(Subject.java:422)
>   at 
> org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1698)
>   ... 40 more
> {noformat}
> This can be reproduced for instance by 
> {noformat}
>   // shameless copy of existing test case
>   @Test
>   public void testFullyReadSingleFileWithSpaces() throws Exception {
> PipelineOptions options = PipelineOptionsFactory.create();
> List> expectedResults = createRandomRecords(3, 10, 
> 0);
> File file = createFileWithData("tmp data.seq", expectedResults);
> HDFSFileSource, IntWritable, Text> source =
> HDFSFileSource.from(
> file.toString(), SequenceFileInputFormat.class, 
> IntWritable.class, Text.class);
> assertEquals(file.length(), source.getEstimatedSizeBytes(null));
> assertThat(expectedResults, containsInAnyOrder(readFromSource(source, 
> options).toArray()));
>   }
> {noformat}
> Changing the implementation slightly to
> {noformat}
> diff --git 
> a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
>  
> b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
> index 2a731fb..df72643 100644
> --- 
> a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
> +++ 
> b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
> @@ -30,7 +30,6 @@ import java.io.ObjectInput;
>  import java.io.ObjectOutput;
>  import java.lang.reflect.InvocationTargetException;
>  import java.lang.reflect.Method;
> -import java.net.URI;
>  import java.security.PrivilegedExceptionAction;
>  import java.util.List;
>  import java.util.ListIterator;
> @@ -337,9 +336,10 @@ public abstract class HDFSFileSource extends 
> BoundedSource {
>  UGIHelper.getBestUGI(username()).doAs(new 
> PrivilegedExceptionAction() {
>@Override
>public Void run() throws Exception {
> -FileSystem fs = FileSystem.get(new URI(filepattern()),
> +final Path pathPattern = new Path(filepattern());
> +FileSystem fs = FileSystem.get(pathPattern.toUri(),
>  
> SerializableConfiguration.newConfiguration(serializableConfiguration()));
> -FileStatus[] fileStatuses = fs.globStatus(new 
> Path(filepattern()));
> +FileStatus[] fileStatuses = fs.globStatus(pathPattern);
>  checkState(
>  fileStatuses != null && fileStatuses.length > 0,
>  "Unable to find any files matching %s", filepattern());
> {noformat}
> seems to be fixing the issue for us.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (BEAM-1571) Flatten on a single input PCollection should have a test associated with it

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

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

ASF GitHub Bot commented on BEAM-1571:
--

GitHub user tweise opened a pull request:

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

[BEAM-1571] fix FlattenPCollectionList with single input translation in 
ApexRunner

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

---
R: @tgroh 
fix for #2120


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

$ git pull https://github.com/tweise/beam BEAM-1571-singleton-flatten

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

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


commit 0166c996039042452b17cee51af20b2d2f32f8a4
Author: Thomas Weise 
Date:   2017-02-28T06:32:28Z

BEAM-1571 fix FlattenPCollectionList with single input translation in 
ApexRunner




> Flatten on a single input PCollection should have a test associated with it
> ---
>
> Key: BEAM-1571
> URL: https://issues.apache.org/jira/browse/BEAM-1571
> Project: Beam
>  Issue Type: Bug
>  Components: sdk-java-core
>Reporter: Thomas Groh
>Assignee: Thomas Groh
>
> The expected behavior is that an output PCollection containing all of the 
> elements of the input PCollection is produced.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[GitHub] beam pull request #2125: [BEAM-1571] fix FlattenPCollectionList with single ...

2017-02-27 Thread tweise
GitHub user tweise opened a pull request:

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

[BEAM-1571] fix FlattenPCollectionList with single input translation in 
ApexRunner

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

---
R: @tgroh 
fix for #2120


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

$ git pull https://github.com/tweise/beam BEAM-1571-singleton-flatten

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

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


commit 0166c996039042452b17cee51af20b2d2f32f8a4
Author: Thomas Weise 
Date:   2017-02-28T06:32:28Z

BEAM-1571 fix FlattenPCollectionList with single input translation in 
ApexRunner




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


[jira] [Commented] (BEAM-1567) hashStream should be closed in PackageUtil#createPackageAttributes()

2017-02-27 Thread JIRA

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

Ismaël Mejía commented on BEAM-1567:


Welcome [~tedyu] ! Nice to see you here, and even better with a contribution, 
awesome!
If you want to work on any of the HBase JIRAs just tell me and I will free it 
for you.
(sorry guys to jump in the thread, just wanted to say Hi to Ted since we had a 
previous conversation about existing HBaseIO in the hbase mailing list).

> hashStream should be closed in PackageUtil#createPackageAttributes()
> 
>
> Key: BEAM-1567
> URL: https://issues.apache.org/jira/browse/BEAM-1567
> Project: Beam
>  Issue Type: Bug
>  Components: runner-dataflow
>Reporter: Ted Yu
>Priority: Minor
>  Labels: newbie, starter
>
> Here is related code:
> {code}
>   OutputStream hashStream = Funnels.asOutputStream(hasher);
> {code}
> hashStream should be closed upon return from the method



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


Jenkins build is back to stable : beam_PostCommit_Java_RunnableOnService_Apex #628

2017-02-27 Thread Apache Jenkins Server
See 




[jira] [Commented] (BEAM-1572) Add per-stage matching of scope in metrics for the DirectRunner

2017-02-27 Thread Aviem Zur (JIRA)

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

Aviem Zur commented on BEAM-1572:
-

Please note that this relates to https://issues.apache.org/jira/browse/BEAM-1344
The ultimate goal is to have a uniform querying logic across all runners.

> Add per-stage matching of scope in metrics for the DirectRunner
> ---
>
> Key: BEAM-1572
> URL: https://issues.apache.org/jira/browse/BEAM-1572
> Project: Beam
>  Issue Type: Bug
>  Components: sdk-java-extensions
>Reporter: Pablo Estrada
>Assignee: Pablo Estrada
>
> e.g. Metrics with scope "Top/Outer/Inner" should be matched by queries with 
> "Top/Outer" scope.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Updated] (BEAM-1563) Flatten Spark runner libraries.

2017-02-27 Thread Aviem Zur (JIRA)

 [ 
https://issues.apache.org/jira/browse/BEAM-1563?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Aviem Zur updated BEAM-1563:

Description: 
Flatten Spark runner libraries into a single package (or two) so that 
everything is private.
See [~kenn] comment: 
https://github.com/apache/beam/pull/2050#discussion_r103136216

  was:
Flatten Spark runner libraries into a single package (or two) so that 
everything is private.
See [~kenn] comment: https://github.com/apache/beam/pull/2050


> Flatten Spark runner libraries.
> ---
>
> Key: BEAM-1563
> URL: https://issues.apache.org/jira/browse/BEAM-1563
> Project: Beam
>  Issue Type: Improvement
>  Components: runner-spark
>Reporter: Amit Sela
>Assignee: Amit Sela
>
> Flatten Spark runner libraries into a single package (or two) so that 
> everything is private.
> See [~kenn] comment: 
> https://github.com/apache/beam/pull/2050#discussion_r103136216



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Updated] (BEAM-1076) DatastoreIO template Options

2017-02-27 Thread Vikas Kedigehalli (JIRA)

 [ 
https://issues.apache.org/jira/browse/BEAM-1076?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Vikas Kedigehalli updated BEAM-1076:

Summary: DatastoreIO template Options  (was: Datastore Delete template)

> DatastoreIO template Options
> 
>
> Key: BEAM-1076
> URL: https://issues.apache.org/jira/browse/BEAM-1076
> Project: Beam
>  Issue Type: New Feature
>  Components: sdk-java-gcp
>Reporter: Vikas Kedigehalli
>Assignee: Vikas Kedigehalli
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


Jenkins build became unstable: beam_PostCommit_Java_RunnableOnService_Apex #627

2017-02-27 Thread Apache Jenkins Server
See 




[1/3] beam-site git commit: Comment Ostatic link as it has been broken for a few weeks

2017-02-27 Thread davor
Repository: beam-site
Updated Branches:
  refs/heads/asf-site 70682771d -> b647ca562


Comment Ostatic link as it has been broken for a few weeks


Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/f1f7063b
Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/f1f7063b
Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/f1f7063b

Branch: refs/heads/asf-site
Commit: f1f7063b7e63d04df5f4cf264ef9df1d85cd02a2
Parents: 7068277
Author: Sourabh Bajaj 
Authored: Mon Feb 27 17:24:58 2017 -0800
Committer: Davor Bonaci 
Committed: Mon Feb 27 18:10:01 2017 -0800

--
 src/_posts/2017-02-01-graduation-media-recap.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
--


http://git-wip-us.apache.org/repos/asf/beam-site/blob/f1f7063b/src/_posts/2017-02-01-graduation-media-recap.md
--
diff --git a/src/_posts/2017-02-01-graduation-media-recap.md 
b/src/_posts/2017-02-01-graduation-media-recap.md
index a6c54a6..ef40916 100644
--- a/src/_posts/2017-02-01-graduation-media-recap.md
+++ b/src/_posts/2017-02-01-graduation-media-recap.md
@@ -33,7 +33,7 @@ include:
 * Datanami: "[Google Lauds Outside Influence on Apache 
Beam](https://www.datanami.com/2017/01/10/google-lauds-outside-influence-apache-beam/)"
 by Alex Woodie.
 * InfoWorld / JavaWorld: "[Apache Beam unifies batch and streaming for big 
data](http://www.infoworld.com/article/3156598/big-data/apache-beam-unifies-batch-and-streaming-for-big-data.html)"
 by Serdar Yegulalp, and republished in 
[JavaWorld](http://www.javaworld.com/article/3156598/big-data/apache-beam-unifies-batch-and-streaming-for-big-data.html).
 * JAXenter: "[In a way, Apache Beam is the glue that connects many big data 
systems together](https://jaxenter.com/apache-beam-interview-131314.html)" by 
Kypriani Sinaris.
-* OStatic: "[Apache Beam Unifies Batch and Streaming Data 
Processing](http://ostatic.com/blog/apache-beam-unifies-batch-and-streaming-data-processing)"
 by Sam Dean.
+* OStatic: "Apache Beam Unifies Batch and Streaming Data Processing" by Sam 
Dean. 
 * Enterprise Apps Today: "[Apache Beam Graduates to Help Define Streaming Data 
Processing](http://www.enterpriseappstoday.com/business-intelligence/data-analytics/apache-beam-graduates-to-help-define-streaming-data-processing.html)"
 by Sean Michael Kerner.
 * The Register: "[Google must be Beaming as Apache announces its new top-level 
projects](http://www.theregister.co.uk/2017/01/10/google_must_be_ibeamiing_as_apache_announces_its_new_top_level_projects/)"
 by Alexander J. Martin.
 * SiliconANGLE: "[Apache Software Foundation announces two more top-level open 
source 
projects](http://siliconangle.com/blog/2017/01/11/apache-software-foundation-announces-2-top-level-projects/)"
 by Mike Wheatley.



[3/3] beam-site git commit: This closes #164

2017-02-27 Thread davor
This closes #164


Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/b647ca56
Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/b647ca56
Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/b647ca56

Branch: refs/heads/asf-site
Commit: b647ca5622a5aadc06d754ef8ae78ad91565b559
Parents: 7068277 1b1757b
Author: Davor Bonaci 
Authored: Mon Feb 27 18:10:22 2017 -0800
Committer: Davor Bonaci 
Committed: Mon Feb 27 18:10:22 2017 -0800

--
 content/blog/2017/02/01/graduation-media-recap.html | 2 +-
 content/feed.xml| 2 +-
 src/_posts/2017-02-01-graduation-media-recap.md | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)
--




[GitHub] beam-site pull request #164: Remove Ostatic link as it has been broken for a...

2017-02-27 Thread asfgit
Github user asfgit closed the pull request at:

https://github.com/apache/beam-site/pull/164


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


[2/3] beam-site git commit: Regenerate website

2017-02-27 Thread davor
Regenerate website


Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/1b1757bb
Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/1b1757bb
Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/1b1757bb

Branch: refs/heads/asf-site
Commit: 1b1757bb9157e733d3aea7c7367f21a8eca10755
Parents: f1f7063
Author: Davor Bonaci 
Authored: Mon Feb 27 18:10:21 2017 -0800
Committer: Davor Bonaci 
Committed: Mon Feb 27 18:10:21 2017 -0800

--
 content/blog/2017/02/01/graduation-media-recap.html | 2 +-
 content/feed.xml| 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/beam-site/blob/1b1757bb/content/blog/2017/02/01/graduation-media-recap.html
--
diff --git a/content/blog/2017/02/01/graduation-media-recap.html 
b/content/blog/2017/02/01/graduation-media-recap.html
index 70e317f..81409a7 100644
--- a/content/blog/2017/02/01/graduation-media-recap.html
+++ b/content/blog/2017/02/01/graduation-media-recap.html
@@ -188,7 +188,7 @@ include:
   Datanami: “https://www.datanami.com/2017/01/10/google-lauds-outside-influence-apache-beam/";>Google
 Lauds Outside Influence on Apache Beam” by Alex Woodie.
   InfoWorld / JavaWorld: “http://www.infoworld.com/article/3156598/big-data/apache-beam-unifies-batch-and-streaming-for-big-data.html";>Apache
 Beam unifies batch and streaming for big data” by Serdar Yegulalp, and 
republished in http://www.javaworld.com/article/3156598/big-data/apache-beam-unifies-batch-and-streaming-for-big-data.html";>JavaWorld.
   JAXenter: “https://jaxenter.com/apache-beam-interview-131314.html";>In a way, Apache 
Beam is the glue that connects many big data systems together” by 
Kypriani Sinaris.
-  OStatic: “http://ostatic.com/blog/apache-beam-unifies-batch-and-streaming-data-processing";>Apache
 Beam Unifies Batch and Streaming Data Processing” by Sam Dean.
+  OStatic: “Apache Beam Unifies Batch and Streaming Data Processing” 
by Sam Dean. 
   Enterprise Apps Today: “http://www.enterpriseappstoday.com/business-intelligence/data-analytics/apache-beam-graduates-to-help-define-streaming-data-processing.html";>Apache
 Beam Graduates to Help Define Streaming Data Processing” by Sean Michael 
Kerner.
   The Register: “http://www.theregister.co.uk/2017/01/10/google_must_be_ibeamiing_as_apache_announces_its_new_top_level_projects/";>Google
 must be Beaming as Apache announces its new top-level projects” by 
Alexander J. Martin.
   SiliconANGLE: “http://siliconangle.com/blog/2017/01/11/apache-software-foundation-announces-2-top-level-projects/";>Apache
 Software Foundation announces two more top-level open source projects” 
by Mike Wheatley.

http://git-wip-us.apache.org/repos/asf/beam-site/blob/1b1757bb/content/feed.xml
--
diff --git a/content/feed.xml b/content/feed.xml
index 5c0cd90..63a513b 100644
--- a/content/feed.xml
+++ b/content/feed.xml
@@ -619,7 +619,7 @@ include:

  • Datanami: “Google Lauds Outside Influence on Apache Beam” by Alex Woodie.
  • InfoWorld / JavaWorld: “Apache Beam unifies batch and streaming for big data” by Serdar Yegulalp, and republished in JavaWorld.
  • ;
  • JAXenter: “In a way, Apache Beam is the glue that connects many big data systems together” by Kypriani Sinaris.
  • -
  • OStatic: “Apache Beam Unifies Batch and Streaming Data Processing” by Sam Dean.
  • +
  • OStatic: “Apache Beam Unifies Batch and Streaming Data Processing” by Sam Dean.
  • Enterprise Apps Today: “Apache Beam Graduates to Help Define Streaming Data Processing” by Sean Michael Kerner.
  • The Register: “Google must be Beaming as Apache announces its new top-level projects” by Alexander J. Martin.

  • [GitHub] beam pull request #2124: Make side inputs a map, rather than embedding the n...

    2017-02-27 Thread robertwb
    GitHub user robertwb opened a pull request:
    
    https://github.com/apache/beam/pull/2124
    
    Make side inputs a map, rather than embedding the name in the message.
    
    The "local" name only make sense in context.
    
    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/robertwb/incubator-beam runner-protos
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2124.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 #2124
    
    
    commit b2c0f93d0f4d63836a79956d75b6dd100e9138ba
    Author: Robert Bradshaw 
    Date:   2017-02-28T01:59:57Z
    
    Make side inputs a map, rather than embedding the name in the message.
    
    The "local" name only make sense in context.
    
    
    
    
    ---
    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 #2123: (Early Draft) Provide implementation of BQIO.Read t...

    2017-02-27 Thread sammcveety
    GitHub user sammcveety opened a pull request:
    
    https://github.com/apache/beam/pull/2123
    
    (Early Draft) Provide implementation of BQIO.Read that does not use Source 
    API
    
    @tgroh (and other interested parties) does this match your understanding of 
    what needs to be built, with respect to structure?  Is the code reuse of 
    *Source reasonable?
    
    @dhalperi FYI
    
    You can merge this pull request into a Git repository by running:
    
    $ git pull https://github.com/sammcveety/incubator-beam 
    add_side_input_bigquery_read
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2123.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 #2123
    
    
    commit 234aca37f5e8fb7d87fce7087e39956ee0320ed5
    Author: Sam McVeety 
    Date:   2017-02-27T03:18:12Z
    
    Provide implementation of BQIO.Read that does not use Source API
    
    
    
    
    ---
    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 #2106: [BEAM-115] More Runner API refinements.

    2017-02-27 Thread asfgit
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/2106
    
    
    ---
    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.
    ---
    
    

    [jira] [Commented] (BEAM-115) Beam Runner API

    2017-02-27 Thread ASF GitHub Bot (JIRA)
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-115?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887038#comment-15887038
     ] 
    
    ASF GitHub Bot commented on BEAM-115:
    -
    
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/2106
    
    
    > Beam Runner API
    > ---
    >
    > Key: BEAM-115
    > URL: https://issues.apache.org/jira/browse/BEAM-115
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: beam-model-runner-api
    >Reporter: Kenneth Knowles
    >Assignee: Kenneth Knowles
    >
    > The PipelineRunner API from the SDK is not ideal for the Beam technical 
    > vision.
    > It has technical limitations:
    >  - The user's DAG (even including library expansions) is never explicitly 
    > represented, so it cannot be analyzed except incrementally, and cannot 
    > necessarily be reconstructed (for example, to display it!).
    >  - The flattened DAG of just primitive transforms isn't well-suited for 
    > display or transform override.
    >  - The TransformHierarchy isn't well-suited for optimizations.
    >  - The user must realistically pre-commit to a runner, and its configuration 
    > (batch vs streaming) prior to graph construction, since the runner will be 
    > modifying the graph as it is built.
    >  - It is fairly language- and SDK-specific.
    > It has usability issues (these are not from intuition, but derived from 
    > actual cases of failure to use according to the design)
    >  - The interleaving of apply() methods in PTransform/Pipeline/PipelineRunner 
    > is confusing.
    >  - The TransformHierarchy, accessible only via visitor traversals, is 
    > cumbersome.
    >  - The staging of construction-time vs run-time is not always obvious.
    > These are just examples. This ticket tracks designing, coming to consensus, 
    > and building an API that more simply and directly supports the technical 
    > vision.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [1/3] beam git commit: Make access_pattern a URN with params.

    2017-02-27 Thread robertwb
    Repository: beam
    Updated Branches:
      refs/heads/master c8dfb851c -> 16736a6b6
    
    
    Make access_pattern a URN with params.
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/a0dcd3f9
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/a0dcd3f9
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/a0dcd3f9
    
    Branch: refs/heads/master
    Commit: a0dcd3f97f48cdca02bae4c371e4e143d00fdba1
    Parents: c8dfb85
    Author: Robert Bradshaw 
    Authored: Fri Feb 24 15:53:39 2017 -0800
    Committer: Robert Bradshaw 
    Committed: Mon Feb 27 17:46:00 2017 -0800
    
    --
     sdks/common/runner-api/src/main/proto/beam_runner_api.proto | 2 +-
     1 file changed, 1 insertion(+), 1 deletion(-)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/a0dcd3f9/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
    --
    diff --git a/sdks/common/runner-api/src/main/proto/beam_runner_api.proto 
    b/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
    index 989e4bb..ad6d0cb 100644
    --- a/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
    +++ b/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
    @@ -591,7 +591,7 @@ message SideInput {
       // The only access pattern intended for Beam, because of its superior
       // performance possibilities, is "urn:beam:sideinput:multimap" (or some such
       // URN)
    -  string access_pattern = 3;
    +  UrnWithParameter access_pattern = 3;
     
       // (Required) The pipeline-scoped id for the FunctionSpec of the UDF that
       // adapts a particular access_pattern to a user-facing view type.
    
    
    

    [3/3] beam git commit: Closes #2106

    2017-02-27 Thread robertwb
    Closes #2106
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/16736a6b
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/16736a6b
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/16736a6b
    
    Branch: refs/heads/master
    Commit: 16736a6b612ba38b19694ae2a23dea7f5fab1563
    Parents: c8dfb85 c04e8ab
    Author: Robert Bradshaw 
    Authored: Mon Feb 27 17:46:02 2017 -0800
    Committer: Robert Bradshaw 
    Committed: Mon Feb 27 17:46:02 2017 -0800
    
    --
     sdks/common/runner-api/src/main/proto/beam_runner_api.proto | 8 +++-
     1 file changed, 3 insertions(+), 5 deletions(-)
    --
    
    
    
    

    [2/3] beam git commit: Clarify side input name and remove redundant pcollection reference.

    2017-02-27 Thread robertwb
    Clarify side input name and remove redundant pcollection reference.
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/c04e8abd
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/c04e8abd
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/c04e8abd
    
    Branch: refs/heads/master
    Commit: c04e8abd41575a4e819c3f18dabff921338f55d8
    Parents: a0dcd3f
    Author: Robert Bradshaw 
    Authored: Fri Feb 24 16:31:05 2017 -0800
    Committer: Robert Bradshaw 
    Committed: Mon Feb 27 17:46:01 2017 -0800
    
    --
     sdks/common/runner-api/src/main/proto/beam_runner_api.proto | 6 ++
     1 file changed, 2 insertions(+), 4 deletions(-)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/c04e8abd/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
    --
    diff --git a/sdks/common/runner-api/src/main/proto/beam_runner_api.proto 
    b/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
    index ad6d0cb..258c278 100644
    --- a/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
    +++ b/sdks/common/runner-api/src/main/proto/beam_runner_api.proto
    @@ -576,12 +576,10 @@ message TimestampTransform {
     // A specification for how to "side input" a PCollection.
     message SideInput {
     
    -  // (Required) A local name for this side input, as embedded in a serialized 
    UDF.
    +  // (Required) A local name for this side input, as interpreted by its
    +  // parent PTransform (and/or its PTransform's UDFs).
       string name = 1;
     
    -  // (Required) The pipeline-scoped unique id of the PCollection to be side 
    input.
    -  string pcollection_id = 2;
    -
       // (Required) URN of the access pattern required by the `view_fn` to present
       // the desired SDK-specific interface to a UDF.
       //
    
    
    

    [jira] [Commented] (BEAM-1572) Add per-stage matching of scope in metrics for the DirectRunner

    2017-02-27 Thread ASF GitHub Bot (JIRA)
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15887030#comment-15887030
     ] 
    
    ASF GitHub Bot commented on BEAM-1572:
    --
    
    GitHub user pabloem opened a pull request:
    
    https://github.com/apache/beam/pull/2122
    
    [BEAM-1572] Adding per-stage level matching of metrics filters.
    
    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.
     - [x] If this contribution is large, please file an Apache
       [Individual Contributor License 
    Agreement](https://www.apache.org/licenses/icla.txt).
    
    ---
    Also fixed a bug in Python metrics.
    
    You can merge this pull request into a Git repository by running:
    
    $ git pull https://github.com/pabloem/incubator-beam metrics-filter
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2122.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 #2122
    
    
    commit bc2a99354406981f75e60b86fd9776d572cb0159
    Author: Pablo 
    Date:   2017-02-28T01:28:26Z
    
    Adding per-stage level matching of metrics filters.
    
    
    
    
    > Add per-stage matching of scope in metrics for the DirectRunner
    > ---
    >
    > Key: BEAM-1572
    > URL: https://issues.apache.org/jira/browse/BEAM-1572
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-extensions
    >Reporter: Pablo Estrada
    >Assignee: Pablo Estrada
    >
    > e.g. Metrics with scope "Top/Outer/Inner" should be matched by queries with 
    > "Top/Outer" scope.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [GitHub] beam pull request #2122: [BEAM-1572] Adding per-stage level matching of metr...

    2017-02-27 Thread pabloem
    GitHub user pabloem opened a pull request:
    
    https://github.com/apache/beam/pull/2122
    
    [BEAM-1572] Adding per-stage level matching of metrics filters.
    
    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.
     - [x] If this contribution is large, please file an Apache
       [Individual Contributor License 
    Agreement](https://www.apache.org/licenses/icla.txt).
    
    ---
    Also fixed a bug in Python metrics.
    
    You can merge this pull request into a Git repository by running:
    
    $ git pull https://github.com/pabloem/incubator-beam metrics-filter
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2122.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 #2122
    
    
    commit bc2a99354406981f75e60b86fd9776d572cb0159
    Author: Pablo 
    Date:   2017-02-28T01:28:26Z
    
    Adding per-stage level matching of metrics filters.
    
    
    
    
    ---
    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.
    ---
    
    

    [jira] [Resolved] (BEAM-1538) Add a fast version of BufferedElementCountingOutputStream

    2017-02-27 Thread Vikas Kedigehalli (JIRA)
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1538?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Vikas Kedigehalli resolved BEAM-1538.
    -
       Resolution: Fixed
    Fix Version/s: Not applicable
    
    No longer need this as we embed this functionality into the IterableCoder 
    directly. 
    
    > Add a fast version of BufferedElementCountingOutputStream
    > -
    >
    > Key: BEAM-1538
    > URL: https://issues.apache.org/jira/browse/BEAM-1538
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: sdk-py
    >Reporter: Vikas Kedigehalli
    >Assignee: Ahmet Altay
    >Priority: Minor
    > Fix For: Not applicable
    >
    >
    > We are currently using python version of the stream which is slow. We need to 
    > implement a Cython version.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [jira] [Comment Edited] (BEAM-1556) Spark executors need to register IO factories

    2017-02-27 Thread Davor Bonaci (JIRA)
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886971#comment-15886971
     ] 
    
    Davor Bonaci edited comment on BEAM-1556 at 2/28/17 1:05 AM:
    -
    
    It certainly is an issue in the other runners as well.
    
    I wouldn't do in the context of a {{FileBasedSource}}. Users should be able to 
    call the {{FileSystem}} API from, say, {{@ProcessElement}} method of a 
    {{DoFn}}. So, I think the registration should be done before any "user code" is 
    invoked.
    
    Doing it in worker startup might not be ideal -- the constructor takes 
    {{PipelineOptions}} as an argument. Since jobs could have different options, it 
    probably needs to happen on a per-task basis, likely at the point the worker 
    receives the task from the master and deserializes {{PipelineOptions}}.
    
    
    was (Author: davor):
    It certainly is an issue in other runners as well.
    
    I wouldn't do in the context of a {{FileBasedSource}}. Users should be able to 
    call the {{FileSystem}} API from, say, {{@ProcessElement}} method of a 
    {{DoFn}}. So, I think the registration should be done before any "user code" is 
    invoked.
    
    Doing it in worker startup might not be ideal -- the constructor takes 
    {{PipelineOptions}} as an argument. Since jobs could have different options, it 
    probably needs to happen on a per-task basis, likely at the point the worker 
    receives the task from the master and deserializes {{PipelineOptions}}.
    
    > Spark executors need to register IO factories
    > -
    >
    > Key: BEAM-1556
    > URL: https://issues.apache.org/jira/browse/BEAM-1556
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-spark
    >Reporter: Frances Perry
    >Assignee: Jean-Baptiste Onofré
    >
    > The Spark executors need to call IOChannelUtils.registerIOFactories(options) 
    > in order to support GCS file and make the default WordCount example work.
    > Context in this thread: 
    > https://lists.apache.org/thread.html/469a139c9eb07e64e514cdea42ab8000678ab743794a090c365205d7@%3Cuser.beam.apache.org%3E
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [jira] [Commented] (BEAM-1556) Spark executors need to register IO factories

    2017-02-27 Thread Davor Bonaci (JIRA)
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1556?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886971#comment-15886971
     ] 
    
    Davor Bonaci commented on BEAM-1556:
    
    
    It certainly is an issue in other runners as well.
    
    I wouldn't do in the context of a {{FileBasedSource}}. Users should be able to 
    call the {{FileSystem}} API from, say, {{@ProcessElement}} method of a 
    {{DoFn}}. So, I think the registration should be done before any "user code" is 
    invoked.
    
    Doing it in worker startup might not be ideal -- the constructor takes 
    {{PipelineOptions}} as an argument. Since jobs could have different options, it 
    probably needs to happen on a per-task basis, likely at the point the worker 
    receives the task from the master and deserializes {{PipelineOptions}}.
    
    > Spark executors need to register IO factories
    > -
    >
    > Key: BEAM-1556
    > URL: https://issues.apache.org/jira/browse/BEAM-1556
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-spark
    >Reporter: Frances Perry
    >Assignee: Jean-Baptiste Onofré
    >
    > The Spark executors need to call IOChannelUtils.registerIOFactories(options) 
    > in order to support GCS file and make the default WordCount example work.
    > Context in this thread: 
    > https://lists.apache.org/thread.html/469a139c9eb07e64e514cdea42ab8000678ab743794a090c365205d7@%3Cuser.beam.apache.org%3E
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    Jenkins build is back to normal : beam_PostCommit_Java_RunnableOnService_Dataflow #2422

    2017-02-27 Thread Apache Jenkins Server
    See 
    
    
    
    

    [jira] [Commented] (BEAM-1567) hashStream should be closed in PackageUtil#createPackageAttributes()

    2017-02-27 Thread ASF GitHub Bot (JIRA)
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886905#comment-15886905
     ] 
    
    ASF GitHub Bot commented on BEAM-1567:
    --
    
    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 should be closed upon return from the method
    
    You can merge this pull request into a Git repository by running:
    
    $ git pull https://github.com/tedyu/beam master
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2121.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 #2121
    
    
    
    
    
    
    > hashStream should be closed in PackageUtil#createPackageAttributes()
    > 
    >
    > Key: BEAM-1567
    > URL: https://issues.apache.org/jira/browse/BEAM-1567
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-dataflow
    >Reporter: Ted Yu
    >Priority: Minor
    >  Labels: newbie, starter
    >
    > Here is related code:
    > {code}
    >   OutputStream hashStream = Funnels.asOutputStream(hasher);
    > {code}
    > hashStream should be closed upon return from the method
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [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 should be closed upon return from the method
    
    You can merge this pull request into a Git repository by running:
    
    $ git pull https://github.com/tedyu/beam master
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2121.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 #2121
    
    
    
    
    
    
    ---
    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.
    ---
    
    

    [jira] [Assigned] (BEAM-1572) Add per-stage matching of scope in metrics for the DirectRunner

    2017-02-27 Thread Davor Bonaci (JIRA)
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1572?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Davor Bonaci reassigned BEAM-1572:
    --
    
    Assignee: Pablo Estrada  (was: Davor Bonaci)
    
    > Add per-stage matching of scope in metrics for the DirectRunner
    > ---
    >
    > Key: BEAM-1572
    > URL: https://issues.apache.org/jira/browse/BEAM-1572
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-extensions
    >Reporter: Pablo Estrada
    >Assignee: Pablo Estrada
    >
    > e.g. Metrics with scope "Top/Outer/Inner" should be matched by queries with 
    > "Top/Outer" scope.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [jira] [Commented] (BEAM-1572) Add per-stage matching of scope in metrics for the DirectRunner

    2017-02-27 Thread Davor Bonaci (JIRA)
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886885#comment-15886885
     ] 
    
    Davor Bonaci commented on BEAM-1572:
    
    
    Thanks [~pabloem].
    
    > Add per-stage matching of scope in metrics for the DirectRunner
    > ---
    >
    > Key: BEAM-1572
    > URL: https://issues.apache.org/jira/browse/BEAM-1572
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-extensions
    >Reporter: Pablo Estrada
    >Assignee: Pablo Estrada
    >
    > e.g. Metrics with scope "Top/Outer/Inner" should be matched by queries with 
    > "Top/Outer" scope.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [jira] [Created] (BEAM-1572) Add per-stage matching of scope in metrics for the DirectRunner

    2017-02-27 Thread Pablo Estrada (JIRA)
    Pablo Estrada created BEAM-1572:
    ---
    
     Summary: Add per-stage matching of scope in metrics for the 
    DirectRunner
     Key: BEAM-1572
     URL: https://issues.apache.org/jira/browse/BEAM-1572
     Project: Beam
      Issue Type: Bug
      Components: sdk-java-extensions
    Reporter: Pablo Estrada
    Assignee: Davor Bonaci
    
    
    e.g. Metrics with scope "Top/Outer/Inner" should be matched by queries with 
    "Top/Outer" scope.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [jira] [Commented] (BEAM-1571) Flatten on a single input PCollection should have a test associated with it

    2017-02-27 Thread ASF GitHub Bot (JIRA)
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1571?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886842#comment-15886842
     ] 
    
    ASF GitHub Bot commented on BEAM-1571:
    --
    
    GitHub user tgroh opened a pull request:
    
    https://github.com/apache/beam/pull/2120
    
    [BEAM-1571] Add a test for FlattenPCollectionList on a singleton list
    
    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).
    
    ---
    This test validates that the output PCollection of a Flatten with a
    single input PCollection contains the contents of the input PCollection.
    
    You can merge this pull request into a Git repository by running:
    
    $ git pull https://github.com/tgroh/beam flatten_singleton_test
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2120.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 #2120
    
    
    
    
    
    
    > Flatten on a single input PCollection should have a test associated with it
    > ---
    >
    > Key: BEAM-1571
    > URL: https://issues.apache.org/jira/browse/BEAM-1571
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-core
    >Reporter: Thomas Groh
    >Assignee: Thomas Groh
    >
    > The expected behavior is that an output PCollection containing all of the 
    > elements of the input PCollection is produced.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [jira] [Commented] (BEAM-1572) Add per-stage matching of scope in metrics for the DirectRunner

    2017-02-27 Thread Pablo Estrada (JIRA)
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886850#comment-15886850
     ] 
    
    Pablo Estrada commented on BEAM-1572:
    -
    
    I'm working on this for both Python and Java.
    
    > Add per-stage matching of scope in metrics for the DirectRunner
    > ---
    >
    > Key: BEAM-1572
    > URL: https://issues.apache.org/jira/browse/BEAM-1572
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-extensions
    >Reporter: Pablo Estrada
    >Assignee: Davor Bonaci
    >
    > e.g. Metrics with scope "Top/Outer/Inner" should be matched by queries with 
    > "Top/Outer" scope.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [jira] [Created] (BEAM-1571) Flatten on a single input PCollection should have a test associated with it

    2017-02-27 Thread Thomas Groh (JIRA)
    Thomas Groh created BEAM-1571:
    -
    
     Summary: Flatten on a single input PCollection should have a test 
    associated with it
     Key: BEAM-1571
     URL: https://issues.apache.org/jira/browse/BEAM-1571
     Project: Beam
      Issue Type: Bug
      Components: sdk-java-core
    Reporter: Thomas Groh
    Assignee: Thomas Groh
    
    
    The expected behavior is that an output PCollection containing all of the 
    elements of the input PCollection is produced.
    
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [GitHub] beam pull request #2120: [BEAM-1571] Add a test for FlattenPCollectionList o...

    2017-02-27 Thread tgroh
    GitHub user tgroh opened a pull request:
    
    https://github.com/apache/beam/pull/2120
    
    [BEAM-1571] Add a test for FlattenPCollectionList on a singleton list
    
    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).
    
    ---
    This test validates that the output PCollection of a Flatten with a
    single input PCollection contains the contents of the input PCollection.
    
    You can merge this pull request into a Git repository by running:
    
    $ git pull https://github.com/tgroh/beam flatten_singleton_test
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2120.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 #2120
    
    
    
    
    
    
    ---
    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.
    ---
    
    

    [jira] [Assigned] (BEAM-1570) broken link on website (ostatic.com)

    2017-02-27 Thread Davor Bonaci (JIRA)
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1570?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Davor Bonaci reassigned BEAM-1570:
    --
    
    Assignee: Sourabh Bajaj  (was: Davor Bonaci)
    
    > broken link on website (ostatic.com)
    > 
    >
    > Key: BEAM-1570
    > URL: https://issues.apache.org/jira/browse/BEAM-1570
    > Project: Beam
    >  Issue Type: Bug
    >  Components: website
    >Reporter: Stephen Sisk
    >Assignee: Sourabh Bajaj
    >
    > I see the following error when running the website test suite locally and on 
    > jenkins: 
    >   *  External link 
    > http://ostatic.com/blog/apache-beam-unifies-batch-and-streaming-data-processing
    >  failed: got a time out (response code 0)
    > It looks like ostatic's website is down. 
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [GitHub] beam-site pull request #166: [BEAM-1552] Add contrib guide comments about us...

    2017-02-27 Thread asfgit
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam-site/pull/166
    
    
    ---
    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.
    ---
    
    

    [jira] [Updated] (BEAM-1567) hashStream should be closed in PackageUtil#createPackageAttributes()

    2017-02-27 Thread Daniel Halperin (JIRA)
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1567?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Daniel Halperin updated BEAM-1567:
    --
    Labels: newbie starter  (was: )
    
    > hashStream should be closed in PackageUtil#createPackageAttributes()
    > 
    >
    > Key: BEAM-1567
    > URL: https://issues.apache.org/jira/browse/BEAM-1567
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-dataflow
    >Reporter: Ted Yu
    >Priority: Minor
    >  Labels: newbie, starter
    >
    > Here is related code:
    > {code}
    >   OutputStream hashStream = Funnels.asOutputStream(hasher);
    > {code}
    > hashStream should be closed upon return from the method
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    

    [3/3] beam-site git commit: This closes #166

    2017-02-27 Thread davor
    This closes #166
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/70682771
    Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/70682771
    Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/70682771
    
    Branch: refs/heads/asf-site
    Commit: 70682771d6bb8ca590e4b04b9943c3b50348496a
    Parents: 2bb1902 582a7f4
    Author: Davor Bonaci 
    Authored: Mon Feb 27 16:18:18 2017 -0800
    Committer: Davor Bonaci 
    Committed: Mon Feb 27 16:18:18 2017 -0800
    
    --
     content/contribute/contribution-guide/index.html | 8 +---
     src/contribute/contribution-guide.md | 8 +---
     2 files changed, 10 insertions(+), 6 deletions(-)
    --
    
    
    
    

    [2/3] beam-site git commit: Regenerate website

    2017-02-27 Thread davor
    Regenerate website
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/582a7f45
    Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/582a7f45
    Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/582a7f45
    
    Branch: refs/heads/asf-site
    Commit: 582a7f452eed30d815cfddc6af331812e63ccf61
    Parents: 2f35897
    Author: Davor Bonaci 
    Authored: Mon Feb 27 16:18:18 2017 -0800
    Committer: Davor Bonaci 
    Committed: Mon Feb 27 16:18:18 2017 -0800
    
    --
     content/contribute/contribution-guide/index.html | 8 +---
     1 file changed, 5 insertions(+), 3 deletions(-)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam-site/blob/582a7f45/content/contribute/contribution-guide/index.html
    --
    diff --git a/content/contribute/contribution-guide/index.html 
    b/content/contribute/contribution-guide/index.html
    index 580b7e6..5f8ec98 100644
    --- a/content/contribute/contribution-guide/index.html
    +++ b/content/contribute/contribution-guide/index.html
    @@ -419,9 +419,9 @@ $ git checkout -b  origin/master
     
     Remember to always use --rebase 
    parameter to avoid extraneous merge commits.
     
    -To push your local, committed changes to your (forked) repository on 
    GitHub, run:
    +Then you can push your local, committed changes to your (forked) repository 
    on GitHub. Since rebase may change that branch’s history, you may need to 
    force push. You’ll run:
     
    -$ git push 
     
    +$ git push 
      --force
     
     
     
    @@ -438,12 +438,14 @@ $ git checkout -b  origin/master
     Once the initial code is complete and the tests pass, it’s time to start 
    the code review process. We review and discuss all code, no matter who authors 
    it. It’s a great way to build community, since you can learn from other 
    developers, and they become familiar with your contribution. It also builds a 
    strong project by encouraging a high quality bar and keeping code consistent 
    throughout the project.
     
     Create a pull request
    -Organize your commits to make your reviewer’s job easier. Use the 
    following command to re-order, squash, edit, or change description of 
    individual commits.
    +Organize your commits to make your reviewer’s job easier. Reviewers 
    normally prefer multiple small pull requests, instead of a single large pull 
    request. Within a pull request, a relatively small number of commits that break 
    the problem into logical steps is preferred. For most pull requests, you’ll 
    squash your changes down to 1 commit. You can use the following command to 
    re-order, squash, edit, or change description of individual commits.
     
     $ git rebase -i 
    origin/master
     
     
     
    +You’ll then push to your branch on GitHub. Note: when updating your 
    commit after pull request feedback and use squash to get back to one commit, 
    you will need to do a force submit to the branch on your repo.
    +
     Navigate to the https://github.com/apache/beam";>Beam GitHub 
    mirror to create a pull request. The title of the pull request should be 
    strictly in the following format:
     
     [BEAM-] 
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05235.html">[jira] [Commented] (BEAM-1552) Beam contrib guide does not fully explain normal dev flow</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1552%5C%29+Beam+contrib+guide+does+not+fully+explain+normal+dev+flow%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1552?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886836#comment-15886836
     ] 
    
    ASF GitHub Bot commented on BEAM-1552:
    --
    
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam-site/pull/166
    
    
    > Beam contrib guide does not fully explain normal dev flow
    > -
    >
    > Key: BEAM-1552
    > URL: https://issues.apache.org/jira/browse/BEAM-1552
    > Project: Beam
    >  Issue Type: Bug
    >  Components: website
    >Reporter: Stephen Sisk
    >Assignee: Stephen Sisk
    >
    > I encountered some issues while starting to work on Beam and it'd be nice to 
    > update the contributor's guide.
    > The contrib guide contains pretty detailed git commands, but:
    > 1. For pushing to your branch, the git commands don't actually work when you 
    > try to use them in real life (you need to force push)
    > 2. For squashing, we tell contributors that they can squash, but we don't 
    > actually say that we expect them to.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05232.html">[1/3] beam-site git commit: Add contrib guide comments about using squash & force push.</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B1%5C%2F3%5C%5D+beam%5C-site+git+commit%5C%3A+Add+contrib+guide+comments+about+using+squash+%5C%26+force+push.%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22davor%22&o=newest&f=1">davor</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Repository: beam-site
    Updated Branches:
      refs/heads/asf-site 2bb190260 -> 70682771d
    
    
    Add contrib guide comments about using squash & force push.
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/2f35897a
    Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/2f35897a
    Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/2f35897a
    
    Branch: refs/heads/asf-site
    Commit: 2f35897a244c5ff0b73f822335a6881ef210f519
    Parents: 2bb1902
    Author: Stephen Sisk 
    Authored: Fri Feb 24 14:44:09 2017 -0800
    Committer: Davor Bonaci 
    Committed: Mon Feb 27 16:17:47 2017 -0800
    
    --
     src/contribute/contribution-guide.md | 8 +---
     1 file changed, 5 insertions(+), 3 deletions(-)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam-site/blob/2f35897a/src/contribute/contribution-guide.md
    --
    diff --git a/src/contribute/contribution-guide.md 
    b/src/contribute/contribution-guide.md
    index 83d92fc..a959feb 100644
    --- a/src/contribute/contribution-guide.md
    +++ b/src/contribute/contribution-guide.md
    @@ -186,9 +186,9 @@ Periodically while you work, and certainly before 
    submitting a pull request, you
     
     Remember to always use `--rebase` parameter to avoid extraneous merge commits.
     
    -To push your local, committed changes to your (forked) repository on GitHub, 
    run:
    +Then you can push your local, committed changes to your (forked) repository on 
    GitHub. Since rebase may change that branch's history, you may need to force 
    push. You'll run:
     
    -   $ git push  
    +   $ git push   --force
     
     ### Testing
     All code should have appropriate unit testing coverage. New code should have 
    new tests in the same contribution. Bug fixes should include a regression test 
    to prevent the issue from reoccurring.
    @@ -201,10 +201,12 @@ For contributions to the Java code, run unit tests 
    locally via Maven. Alternativ
     Once the initial code is complete and the tests pass, it’s time to start the 
    code review process. We review and discuss all code, no matter who authors it. 
    It’s a great way to build community, since you can learn from other 
    developers, and they become familiar with your contribution. It also builds a 
    strong project by encouraging a high quality bar and keeping code consistent 
    throughout the project.
     
     ### Create a pull request
    -Organize your commits to make your reviewer’s job easier. Use the following 
    command to re-order, squash, edit, or change description of individual commits.
    +Organize your commits to make your reviewer’s job easier. Reviewers normally 
    prefer multiple small pull requests, instead of a single large pull request. 
    Within a pull request, a relatively small number of commits that break the 
    problem into logical steps is preferred. For most pull requests, you'll squash 
    your changes down to 1 commit. You can use the following command to re-order, 
    squash, edit, or change description of individual commits.
     
     $ git rebase -i origin/master
     
    +You'll then push to your branch on GitHub. Note: when updating your commit 
    after pull request feedback and use squash to get back to one commit, you will 
    need to do a force submit to the branch on your repo.
    +
     Navigate to the [Beam GitHub mirror](https://github.com/apache/beam) to create 
    a pull request. The title of the pull request should be strictly in the 
    following format:
     
    [BEAM-] 
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05231.html">[jira] [Created] (BEAM-1570) broken link on website (ostatic.com)</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCreated%5C%5D+%5C%28BEAM%5C-1570%5C%29+broken+link+on+website+%5C%28ostatic.com%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Stephen+Sisk+%5C%28JIRA%5C%29%22&o=newest&f=1">Stephen Sisk (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Stephen Sisk created BEAM-1570:
    --
    
     Summary: broken link on website (ostatic.com)
     Key: BEAM-1570
     URL: https://issues.apache.org/jira/browse/BEAM-1570
     Project: Beam
      Issue Type: Bug
      Components: website
    Reporter: Stephen Sisk
    Assignee: Davor Bonaci
    
    
    I see the following error when running the website test suite locally and on 
    jenkins: 
    
      *  External link 
    http://ostatic.com/blog/apache-beam-unifies-batch-and-streaming-data-processing 
    failed: got a time out (response code 0)
    
    It looks like ostatic's website is down. 
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05229.html">[GitHub] beam-site pull request #167: [BEAM-1559] Reference HBase support in the docu...</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5BGitHub%5C%5D+beam%5C-site+pull+request+%23167%5C%3A+%5C%5BBEAM%5C-1559%5C%5D+Reference+HBase+support+in+the+docu...%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22asfgit%22&o=newest&f=1">asfgit</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam-site/pull/167
    
    
    ---
    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.
    ---
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05230.html">[jira] [Commented] (BEAM-1559) Reference HBase support in the documentation</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1559%5C%29+Reference+HBase+support+in+the+documentation%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1559?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886827#comment-15886827
     ] 
    
    ASF GitHub Bot commented on BEAM-1559:
    --
    
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam-site/pull/167
    
    
    > Reference HBase support in the documentation
    > 
    >
    > Key: BEAM-1559
    > URL: https://issues.apache.org/jira/browse/BEAM-1559
    > Project: Beam
    >  Issue Type: Task
    >  Components: project-management
    >Reporter: Tom Haines
    >Assignee: Jean-Baptiste Onofré
    >Priority: Minor
    >  Labels: documentation
    >
    > The HBase connector is not referenced in the docs.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05227.html">[2/3] beam-site git commit: Regenerate website</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B2%5C%2F3%5C%5D+beam%5C-site+git+commit%5C%3A+Regenerate+website%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22davor%22&o=newest&f=1">davor</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Regenerate website
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/f0af7937
    Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/f0af7937
    Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/f0af7937
    
    Branch: refs/heads/asf-site
    Commit: f0af7937291e92e703af6b52fff10c257cbddf01
    Parents: 1f44221
    Author: Davor Bonaci 
    Authored: Mon Feb 27 16:12:13 2017 -0800
    Committer: Davor Bonaci 
    Committed: Mon Feb 27 16:12:13 2017 -0800
    
    --
     content/documentation/programming-guide/index.html | 1 +
     content/documentation/sdks/java/index.html | 1 +
     2 files changed, 2 insertions(+)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam-site/blob/f0af7937/content/documentation/programming-guide/index.html
    --
    diff --git a/content/documentation/programming-guide/index.html 
    b/content/documentation/programming-guide/index.html
    index 75a04e5..a6403e0 100644
    --- a/content/documentation/programming-guide/index.html
    +++ b/content/documentation/programming-guide/index.html
    @@ -1316,6 +1316,7 @@ tree, [2]
     https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/io";>Google
     Cloud PubSub
       
       
    +https://github.com/apache/beam/tree/master/sdks/java/io/hbase";>Apache 
    HBase
     https://github.com/apache/beam/tree/master/sdks/java/io/mongodb";>MongoDB
     https://github.com/apache/beam/tree/master/sdks/java/io/jdbc";>JDBC
     https://github.com/apache/beam/tree/master/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery";>Google
     BigQuery
    
    http://git-wip-us.apache.org/repos/asf/beam-site/blob/f0af7937/content/documentation/sdks/java/index.html
    --
    diff --git a/content/documentation/sdks/java/index.html 
    b/content/documentation/sdks/java/index.html
    index ce0a6be..fc3123e 100644
    --- a/content/documentation/sdks/java/index.html
    +++ b/content/documentation/sdks/java/index.html
    @@ -169,6 +169,7 @@
     
       Amazon Kinesis
       Apache Hadoop’s FileInputFormat 
    in Hadoop Distributed File System (HDFS)
    +  Apache HBase
       Apache Kafka
       Avro Files
       Google BigQuery
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05228.html">[1/3] beam-site git commit: BEAM-1559 add HBase docs</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B1%5C%2F3%5C%5D+beam%5C-site+git+commit%5C%3A+BEAM%5C-1559+add+HBase+docs%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22davor%22&o=newest&f=1">davor</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Repository: beam-site
    Updated Branches:
      refs/heads/asf-site 7ae78e1d0 -> 2bb190260
    
    
    BEAM-1559 add HBase docs
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/1f44221c
    Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/1f44221c
    Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/1f44221c
    
    Branch: refs/heads/asf-site
    Commit: 1f44221c58f85c00d8bcb9e957d8d63ee74a472c
    Parents: 7ae78e1
    Author: Tom Haines 
    Authored: Mon Feb 27 15:14:42 2017 +0800
    Committer: Tom Haines 
    Committed: Mon Feb 27 15:14:42 2017 +0800
    
    --
     src/documentation/programming-guide.md | 1 +
     src/documentation/sdks/java.md | 1 +
     2 files changed, 2 insertions(+)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam-site/blob/1f44221c/src/documentation/programming-guide.md
    --
    diff --git a/src/documentation/programming-guide.md 
    b/src/documentation/programming-guide.md
    index 8d6cf0a..f70d969 100644
    --- a/src/documentation/programming-guide.md
    +++ b/src/documentation/programming-guide.md
    @@ -1014,6 +1014,7 @@ See the language specific source code directories for the 
    Beam supported I/O API
     https://github.com/apache/beam/blob/master/sdks/java/core/src/main/java/org/apache/beam/sdk/io";>Google
     Cloud PubSub
       
       
    +https://github.com/apache/beam/tree/master/sdks/java/io/hbase";>Apache 
    HBase
     https://github.com/apache/beam/tree/master/sdks/java/io/mongodb";>MongoDB
     https://github.com/apache/beam/tree/master/sdks/java/io/jdbc";>JDBC
     https://github.com/apache/beam/tree/master/sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/bigquery";>Google
     BigQuery
    
    http://git-wip-us.apache.org/repos/asf/beam-site/blob/1f44221c/src/documentation/sdks/java.md
    --
    diff --git a/src/documentation/sdks/java.md b/src/documentation/sdks/java.md
    index 540eec4..1a3d856 100644
    --- a/src/documentation/sdks/java.md
    +++ b/src/documentation/sdks/java.md
    @@ -25,6 +25,7 @@ The Java SDK supports all features currently supported by the 
    Beam model.
     
     * Amazon Kinesis
     * Apache Hadoop's `FileInputFormat` in Hadoop Distributed File System (HDFS)
    +* Apache HBase
     * Apache Kafka
     * Avro Files
     * Google BigQuery
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05226.html">[3/3] beam-site git commit: This closes #167</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B3%5C%2F3%5C%5D+beam%5C-site+git+commit%5C%3A+This+closes+%23167%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22davor%22&o=newest&f=1">davor</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    This closes #167
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam-site/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam-site/commit/2bb19026
    Tree: http://git-wip-us.apache.org/repos/asf/beam-site/tree/2bb19026
    Diff: http://git-wip-us.apache.org/repos/asf/beam-site/diff/2bb19026
    
    Branch: refs/heads/asf-site
    Commit: 2bb1902602d9f0ca8d812fec9182944b907c393e
    Parents: 7ae78e1 f0af793
    Author: Davor Bonaci 
    Authored: Mon Feb 27 16:12:13 2017 -0800
    Committer: Davor Bonaci 
    Committed: Mon Feb 27 16:12:13 2017 -0800
    
    --
     content/documentation/programming-guide/index.html | 1 +
     content/documentation/sdks/java/index.html | 1 +
     src/documentation/programming-guide.md | 1 +
     src/documentation/sdks/java.md | 1 +
     4 files changed, 4 insertions(+)
    --
    
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05225.html">svn commit: r18496 - /dev/beam/KEYS</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22svn+commit%5C%3A+r18496+%5C-+%5C%2Fdev%5C%2Fbeam%5C%2FKEYS%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22altay%22&o=newest&f=1">altay</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Author: altay
    Date: Tue Feb 28 00:08:57 2017
    New Revision: 18496
    
    Log:
    Add al...@apache.org signing certificate
    
    Modified:
    dev/beam/KEYS
    
    Modified: dev/beam/KEYS
    ==
    --- dev/beam/KEYS (original)
    +++ dev/beam/KEYS Tue Feb 28 00:08:57 2017
    @@ -197,3 +197,61 @@ Me5hxczj1cbOY9i4EgMYkxE7hmrh128m/rPQ9ACm
     taO8cg/VyiDlV7BqlFkmZqscodft/no=
     =odEd
     -END PGP PUBLIC KEY BLOCK-
    +pub   4096R/6096FA00 2017-02-27
    +uid  Ahmet Altay (CODE SIGNING KEY) 
    +sig 36096FA00 2017-02-27  Ahmet Altay (CODE SIGNING KEY) 
    
    +sub   4096R/A969A06A 2017-02-27
    +sig  6096FA00 2017-02-27  Ahmet Altay (CODE SIGNING KEY) 
    
    +
    +-BEGIN PGP PUBLIC KEY BLOCK-
    +Version: GnuPG v1
    +
    +mQINBFi0tOMBEACv5s3rtN1gr9+Arux39vh6fgNPUE7PWl8yW3fhxIxQZESob+hM
    +2fPydZRSPYQanpmA7zKYwI0kN7xP7KU6ZiainqlOSKYXHqojye3M2V0R72XvHCD4
    +1JqqY0pCh+HCrTSTKZU6pyNSonHQrAGGCAVhvARRT9ezi3/TXfaloz9DclbwXvRK
    +wtkZDv/GPZQ6+QQ++kd89YRXpfD2KZdIx4JgmFQCAsmkKeRZmrz6wBlQP7gHeBfw
    +2ekUtRnWabFhqni4X12156/7V0swaWtLMVikdSy6jgsp40lY8ChHq2lc5cBo+CUm
    +O3ryjJO8sdJed0LY/FvlUgUhS53rzPzDIwJGQWXAu1p0ltqLT7zmv6zQM5ox6W6Z
    +yG3FryM/a2Pxp6i3jeAVJTNFJLv6YQNOwHxc7qSmYFudBKo2985kcvXIl9KVNTia
    +M5aIf+b8IYcs+gGJONPcOFlTftXgrRIUofBTblXFNoFByS9TQwpNKJmd7sjlr+kJ
    +RnnjsiHUPf/G4/42bE0mTnLxxNXBCGWCQ/e12e4HNG+3MryfZcjAmjAfriDS1Dep
    +lSrV5fbOEaTm8jwxCJdMn7culOmD45kJqORj0J71+TBMKX8ABfA+F2SxA9jgLyKI
    +oZcIZYTBwiKfs4xi6lyaA0DZkYJ0w08qYQYabwLevwFpAmYkZiiK7drQsQARAQAB
    +tDFBaG1ldCBBbHRheSAoQ09ERSBTSUdOSU5HIEtFWSkgPGFsdGF5QGFwYWNoZS5v
    +cmc+iQI3BBMBAgAhAhsDAh4BAheABQJYtLXQBQsJCAcDBRUKCQgLBRYCAwEAAAoJ
    +ENYucUFglvoA3RkQAKhNCw5IsbIcmO9H2r0pxQoEpGGpSJWsGgnLJ0VAoRlzC5UK
    +waW61ikZ2K2/LGXkIC1ZMnrBDpTbHi5SikmEWKece2Yl5/vhrhigfKpRNLYntrSE
    +0DckXx26KrZY6b4ih3xtykTorD2qJu2C2S2kN3Qo1rk7kyTITHqMzOk0JDIZmoDM
    +Q683/1/ZnFAcMy+M4RDIOv2kHfAbbupdDskAj8rfL0mWXcmXMU1EXAfZNm0fY1fy
    +MhKCEqfrMFABNwUI8cBAOkjLDdx/hTRHlquuEm/Cgb2MucycVeAcI1V2KATGrH7C
    +/q0yktfdgpuuJNyNUCNagkY+2W4egWERRU1Ol+MZVf1GAIK5uJ18sqhkpnCCWSC4
    +loTi8BypPkOb8lrp+B/QmTwpF5GtMp9s2v0hn46E5Z56j+XBcWkWTQ95/iUyC9Dy
    +YwwEcfU0XKW0pLrdyTgIgzCdqfWQshrS5gLe2/Ji24E2Wx5MDh56va8HJOmg6cuj
    +0CK+xvQSB2yiwqoYnT4fczaV76gq/KHxM9VZqxyN2SHiBFYVbNieuCfDOMc1Ja3H
    +BlvKaMqXLNhnJylGAzYDlAMJLzsAJYn+1dDEJdEzDn/z9xfkf1SCJs1G4pFOSFmd
    +jYGpEju/vafEsI0PBkjs1LS6A5irxicKcdhkJsXbPnaaGj4OYGPug4OqtKEbuQIN
    +BFi0tOMBEACf0FQ0DTZLVJLQOXCue3XHe4/psJo/iAJ5KfKGBHwbb7iq6Trm3JWF
    +V7C/hY4JU/VgTEebrPEdj2osbRTq9mSg5RzS+6gHiUgqie5exImAnm1Y+GgWd1K+
    +HsJgHQgaQJshklbD+AUy5PfbX42/R9ZWb9khfFYBitQwiUzyrSOKZhRudKiVQOf1
    +2CTFLXISN8vZMY5YZKHfUQ2SsMoECnbUPHjnOJEv1QbxeOnwBLEDrybah15bndXR
    +Rmh1Oj+jwAhDdpiohg2pEN/FsokczR1UASeL81SogBvpivmOcPwl16tF96wMPHUK
    +6Tt3yCFo/uu4HyVftWcRunbhTFu56SSod3ODAXhTNfO77UUpE/I09KxAbYxJXdnz
    +1Qj4XDTMgid6NBLl2QHCaxEgkucjkFr9JgY5rt+9BGOKWBI8ZPhGQGYQ/XJLMX2Z
    +xJuqHyEY2hT3SlwjZh1SBD1kLJRk/46m5I/vJFQpkfmxlXmxHPXbDi0ffjwCf8dn
    +S1ONF1NAxtK9FLtWy/09k9jRe0VgkIMnEHatEy07INtOW0XpzHIflMc/thR249bF
    +ZQL9MVCLpZr+4QsPdh9o9Vz99rNgT1hB50YtIErr6VsfOCdkWJ+qNmx9wMnMGMpW
    +bdQvkbWfAuGuZjFb2wMC3e1sWsez+dcXzEmGZ1WXARThdR63pC/g6QARAQABiQIf
    +BBgBAgAJBQJYtLTjAhsMAAoJENYucUFglvoAjsYP/3aT5yGBeMMTkTbp2ulkrlM+
    +v4s3PZB4+OQygqjvWLV3TdJp99xO+qSM+2RVfPtEudVoIychEYV3GyRKWeUyEc3b
    +Ab/ZlXUo2R7y/FmlO5Mq9Sx4F58mAvRdaxTh6Qs/f6sUI3zCPTKYq+//mlfv3aty
    +kSLR3cU1GPqOdcFPc84PIQsOTkiNkCvIRl4ixTv8T+oQ99reOL7UXrBIDv6vw04/
    +F0x7JM5htpoFvlcvWg3BsVmT276o4FMvayAVL8rXUeUFJjVOdgGixAqJg0b0k+tW
    +5l/fjCd4lY+riL0iTS5Hw7x8xz2wbhvCDV/laPsSqjshPgluAqjHKnXmp6E3tmIy
    +vUwOQK+1t++/AVdqriofMTp4YhDhrnOkck2AcZ0Hn0v6k/N94GVoIzq0tN69iyoo
    +YS1gm3dkWYYNoGM8WNJeDkhFOWSKL1DigJuMxGCL6ENt+qxioyNo7iAMDdRfMtwU
    +YoShNSaCmQG04dwA0wo+EVaMbOKkF/UuhXj6QURDzoevmvTRx5ipuUFKGZjvAy63
    +viAojjzGCJ7mmpSpRjfPpJ95Dlpa9Kyfw2ag0L+/OYtWVfpWoWLZj9hFfv/Y4kXq
    +V3m2QtLzHBUqN+/FTzUyDASO8kO4J0OaBB6iOTlZUrt7XfycGn9vnqBBC8MGqD8w
    +ic/pJUsMBOm0ADPVB7YO
    +=5xpx
    +-END PGP PUBLIC KEY BLOCK-
    
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05224.html">[jira] [Assigned] (BEAM-1569) HDFSFileSource: Unable to read from filePattern with spaces in path</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BAssigned%5C%5D+%5C%28BEAM%5C-1569%5C%29+HDFSFileSource%5C%3A+Unable+to+read+from+filePattern+with+spaces+in+path%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Davor+Bonaci+%5C%28JIRA%5C%29%22&o=newest&f=1">Davor Bonaci (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1569?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Davor Bonaci reassigned BEAM-1569:
    --
    
    Assignee: Michael Luckey  (was: Davor Bonaci)
    
    > HDFSFileSource: Unable to read from filePattern with spaces in path
    > ---
    >
    > Key: BEAM-1569
    > URL: https://issues.apache.org/jira/browse/BEAM-1569
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-core
    >Reporter: Michael Luckey
    >Assignee: Michael Luckey
    >
    > After the merge of the changes introduced with 
    > https://issues.apache.org/jira/browse/BEAM-1497 we are unable to read from 
    > files containing spaces in path. We encounter following stack trace
    > {noformat}
    > java.lang.reflect.UndeclaredThrowableException
    >   at 
    > org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1713)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource.validate(HDFSFileSource.java:337)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource.createReader(HDFSFileSource.java:329)
    >   at 
    > org.apache.beam.sdk.testing.SourceTestUtils.readFromSource(SourceTestUtils.java:138)
    > Caused by: java.net.URISyntaxException: Illegal character in path at index 
    > 77: 
    > /var/folders/1t/s9pcmfj50nxbt68h3_2z_5wcgn/T/junit6887354597440386901/tmp 
    > data.seq
    >   at java.net.URI$Parser.fail(URI.java:2848)
    >   at java.net.URI$Parser.checkChars(URI.java:3021)
    >   at java.net.URI$Parser.parseHierarchical(URI.java:3105)
    >   at java.net.URI$Parser.parse(URI.java:3063)
    >   at java.net.URI.(URI.java:588)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:340)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:337)
    >   at java.security.AccessController.doPrivileged(Native Method)
    >   at javax.security.auth.Subject.doAs(Subject.java:422)
    >   at 
    > org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1698)
    >   ... 40 more
    > {noformat}
    > This can be reproduced for instance by 
    > {noformat}
    >   // shameless copy of existing test case
    >   @Test
    >   public void testFullyReadSingleFileWithSpaces() throws Exception {
    > PipelineOptions options = PipelineOptionsFactory.create();
    > List> expectedResults = createRandomRecords(3, 10, 
    > 0);
    > File file = createFileWithData("tmp data.seq", expectedResults);
    > HDFSFileSource, IntWritable, Text> source =
    > HDFSFileSource.from(
    > file.toString(), SequenceFileInputFormat.class, 
    > IntWritable.class, Text.class);
    > assertEquals(file.length(), source.getEstimatedSizeBytes(null));
    > assertThat(expectedResults, containsInAnyOrder(readFromSource(source, 
    > options).toArray()));
    >   }
    > {noformat}
    > Changing the implementation slightly to
    > {noformat}
    > diff --git 
    > a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    >  
    > b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > index 2a731fb..df72643 100644
    > --- 
    > a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > +++ 
    > b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > @@ -30,7 +30,6 @@ import java.io.ObjectInput;
    >  import java.io.ObjectOutput;
    >  import java.lang.reflect.InvocationTargetException;
    >  import java.lang.reflect.Method;
    > -import java.net.URI;
    >  import java.security.PrivilegedExceptionAction;
    >  import java.util.List;
    >  import java.util.ListIterator;
    > @@ -337,9 +336,10 @@ public abstract class HDFSFileSource extends 
    > BoundedSource {
    >  UGIHelper.getBestUGI(username()).doAs(new 
    > PrivilegedExceptionAction() {
    >@Override
    >public Void run() throws Exception {
    > -FileSystem fs = FileSystem.get(new URI(filepattern()),
    > +final Path pathPattern = new Path(filepattern());
    > +FileSystem fs = FileSystem.get(pathPattern.toUri(),
    >  
    > SerializableConfiguration.newConfiguration(serializableConfiguration()));
    > -FileStatus[] fileStatuses = fs.globStatus(new 
    > Path(filepattern()));
    > +FileStatus[] fileStatuses = fs.globStatus(pathPattern);
    >  checkState(
    >  fileStatuses != null && fileStatuses.length > 0,
    >  "Unable to find any files matching %s", filepattern());
    > {noformat}
    > seems to be fixing the issue for us.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05223.html">[jira] [Commented] (BEAM-1569) HDFSFileSource: Unable to read from filePattern with spaces in path</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1569%5C%29+HDFSFileSource%5C%3A+Unable+to+read+from+filePattern+with+spaces+in+path%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Davor+Bonaci+%5C%28JIRA%5C%29%22&o=newest&f=1">Davor Bonaci (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886822#comment-15886822
     ] 
    
    Davor Bonaci commented on BEAM-1569:
    
    
    It does. Just make sure the PR includes the test case to make sure this doesn't 
    regress.
    
    (Normally, I wouldn't expect the filepattern to directly map to the URI; some 
    pre-processing is generally needed. Going forward, match() API in the new Beam 
    FileSystem API would do this. So, some change needed in the future here.)
    
    > HDFSFileSource: Unable to read from filePattern with spaces in path
    > ---
    >
    > Key: BEAM-1569
    > URL: https://issues.apache.org/jira/browse/BEAM-1569
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-core
    >Reporter: Michael Luckey
    >Assignee: Davor Bonaci
    >
    > After the merge of the changes introduced with 
    > https://issues.apache.org/jira/browse/BEAM-1497 we are unable to read from 
    > files containing spaces in path. We encounter following stack trace
    > {noformat}
    > java.lang.reflect.UndeclaredThrowableException
    >   at 
    > org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1713)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource.validate(HDFSFileSource.java:337)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource.createReader(HDFSFileSource.java:329)
    >   at 
    > org.apache.beam.sdk.testing.SourceTestUtils.readFromSource(SourceTestUtils.java:138)
    > Caused by: java.net.URISyntaxException: Illegal character in path at index 
    > 77: 
    > /var/folders/1t/s9pcmfj50nxbt68h3_2z_5wcgn/T/junit6887354597440386901/tmp 
    > data.seq
    >   at java.net.URI$Parser.fail(URI.java:2848)
    >   at java.net.URI$Parser.checkChars(URI.java:3021)
    >   at java.net.URI$Parser.parseHierarchical(URI.java:3105)
    >   at java.net.URI$Parser.parse(URI.java:3063)
    >   at java.net.URI.(URI.java:588)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:340)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:337)
    >   at java.security.AccessController.doPrivileged(Native Method)
    >   at javax.security.auth.Subject.doAs(Subject.java:422)
    >   at 
    > org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1698)
    >   ... 40 more
    > {noformat}
    > This can be reproduced for instance by 
    > {noformat}
    >   // shameless copy of existing test case
    >   @Test
    >   public void testFullyReadSingleFileWithSpaces() throws Exception {
    > PipelineOptions options = PipelineOptionsFactory.create();
    > List> expectedResults = createRandomRecords(3, 10, 
    > 0);
    > File file = createFileWithData("tmp data.seq", expectedResults);
    > HDFSFileSource, IntWritable, Text> source =
    > HDFSFileSource.from(
    > file.toString(), SequenceFileInputFormat.class, 
    > IntWritable.class, Text.class);
    > assertEquals(file.length(), source.getEstimatedSizeBytes(null));
    > assertThat(expectedResults, containsInAnyOrder(readFromSource(source, 
    > options).toArray()));
    >   }
    > {noformat}
    > Changing the implementation slightly to
    > {noformat}
    > diff --git 
    > a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    >  
    > b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > index 2a731fb..df72643 100644
    > --- 
    > a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > +++ 
    > b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > @@ -30,7 +30,6 @@ import java.io.ObjectInput;
    >  import java.io.ObjectOutput;
    >  import java.lang.reflect.InvocationTargetException;
    >  import java.lang.reflect.Method;
    > -import java.net.URI;
    >  import java.security.PrivilegedExceptionAction;
    >  import java.util.List;
    >  import java.util.ListIterator;
    > @@ -337,9 +336,10 @@ public abstract class HDFSFileSource extends 
    > BoundedSource {
    >  UGIHelper.getBestUGI(username()).doAs(new 
    > PrivilegedExceptionAction() {
    >@Override
    >public Void run() throws Exception {
    > -FileSystem fs = FileSystem.get(new URI(filepattern()),
    > +final Path pathPattern = new Path(filepattern());
    > +FileSystem fs = FileSystem.get(pathPattern.toUri(),
    >  
    > SerializableConfiguration.newConfiguration(serializableConfiguration()));
    > -FileStatus[] fileStatuses = fs.globStatus(new 
    > Path(filepattern()));
    > +FileStatus[] fileStatuses = fs.globStatus(pathPattern);
    >  checkState(
    >  fileStatuses != null && fileStatuses.length > 0,
    >  "Unable to find any files matching %s", filepattern());
    > {noformat}
    > seems to be fixing the issue for us</pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05222.html">[jira] [Commented] (BEAM-1569) HDFSFileSource: Unable to read from filePattern with spaces in path</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1569%5C%29+HDFSFileSource%5C%3A+Unable+to+read+from+filePattern+with+spaces+in+path%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Michael+Luckey+%5C%28JIRA%5C%29%22&o=newest&f=1">Michael Luckey (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886797#comment-15886797
     ] 
    
    Michael Luckey commented on BEAM-1569:
    --
    
    [~davor] If the suggested changes do make sense, i could prepare a PR. I just 
    wait for the thoughts of the others.
    
    > HDFSFileSource: Unable to read from filePattern with spaces in path
    > ---
    >
    > Key: BEAM-1569
    > URL: https://issues.apache.org/jira/browse/BEAM-1569
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-core
    >Reporter: Michael Luckey
    >Assignee: Davor Bonaci
    >
    > After the merge of the changes introduced with 
    > https://issues.apache.org/jira/browse/BEAM-1497 we are unable to read from 
    > files containing spaces in path. We encounter following stack trace
    > {noformat}
    > java.lang.reflect.UndeclaredThrowableException
    >   at 
    > org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1713)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource.validate(HDFSFileSource.java:337)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource.createReader(HDFSFileSource.java:329)
    >   at 
    > org.apache.beam.sdk.testing.SourceTestUtils.readFromSource(SourceTestUtils.java:138)
    > Caused by: java.net.URISyntaxException: Illegal character in path at index 
    > 77: 
    > /var/folders/1t/s9pcmfj50nxbt68h3_2z_5wcgn/T/junit6887354597440386901/tmp 
    > data.seq
    >   at java.net.URI$Parser.fail(URI.java:2848)
    >   at java.net.URI$Parser.checkChars(URI.java:3021)
    >   at java.net.URI$Parser.parseHierarchical(URI.java:3105)
    >   at java.net.URI$Parser.parse(URI.java:3063)
    >   at java.net.URI.(URI.java:588)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:340)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:337)
    >   at java.security.AccessController.doPrivileged(Native Method)
    >   at javax.security.auth.Subject.doAs(Subject.java:422)
    >   at 
    > org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1698)
    >   ... 40 more
    > {noformat}
    > This can be reproduced for instance by 
    > {noformat}
    >   // shameless copy of existing test case
    >   @Test
    >   public void testFullyReadSingleFileWithSpaces() throws Exception {
    > PipelineOptions options = PipelineOptionsFactory.create();
    > List> expectedResults = createRandomRecords(3, 10, 
    > 0);
    > File file = createFileWithData("tmp data.seq", expectedResults);
    > HDFSFileSource, IntWritable, Text> source =
    > HDFSFileSource.from(
    > file.toString(), SequenceFileInputFormat.class, 
    > IntWritable.class, Text.class);
    > assertEquals(file.length(), source.getEstimatedSizeBytes(null));
    > assertThat(expectedResults, containsInAnyOrder(readFromSource(source, 
    > options).toArray()));
    >   }
    > {noformat}
    > Changing the implementation slightly to
    > {noformat}
    > diff --git 
    > a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    >  
    > b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > index 2a731fb..df72643 100644
    > --- 
    > a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > +++ 
    > b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > @@ -30,7 +30,6 @@ import java.io.ObjectInput;
    >  import java.io.ObjectOutput;
    >  import java.lang.reflect.InvocationTargetException;
    >  import java.lang.reflect.Method;
    > -import java.net.URI;
    >  import java.security.PrivilegedExceptionAction;
    >  import java.util.List;
    >  import java.util.ListIterator;
    > @@ -337,9 +336,10 @@ public abstract class HDFSFileSource extends 
    > BoundedSource {
    >  UGIHelper.getBestUGI(username()).doAs(new 
    > PrivilegedExceptionAction() {
    >@Override
    >public Void run() throws Exception {
    > -FileSystem fs = FileSystem.get(new URI(filepattern()),
    > +final Path pathPattern = new Path(filepattern());
    > +FileSystem fs = FileSystem.get(pathPattern.toUri(),
    >  
    > SerializableConfiguration.newConfiguration(serializableConfiguration()));
    > -FileStatus[] fileStatuses = fs.globStatus(new 
    > Path(filepattern()));
    > +FileStatus[] fileStatuses = fs.globStatus(pathPattern);
    >  checkState(
    >  fileStatuses != null && fileStatuses.length > 0,
    >  "Unable to find any files matching %s", filepattern());
    > {noformat}
    > seems to be fixing the issue for us.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05221.html">[jira] [Commented] (BEAM-1534) Create a dockerized developer environment for Beam</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1534%5C%29+Create+a+dockerized+developer+environment+for+Beam%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Nitin+Lamba+%5C%28JIRA%5C%29%22&o=newest&f=1">Nitin Lamba (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1534?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886752#comment-15886752
     ] 
    
    Nitin Lamba commented on BEAM-1534:
    ---
    
    [~ekremaksoy],
    
    Since you're planning to work on this, please go-ahead and assign this JIRA to 
    yourself. Thanks
    
    > Create a dockerized developer environment for Beam
    > --
    >
    > Key: BEAM-1534
    > URL: https://issues.apache.org/jira/browse/BEAM-1534
    > Project: Beam
    >  Issue Type: New Feature
    >  Components: examples-java
    >Reporter: Nitin Lamba
    >Assignee: Frances Perry
    >
    > This will help create a repeatable developer environment setup.
    > Other Apache projects can be used as a reference:
    > https://github.com/apache/ambari/tree/trunk/dev-support/docker
    > https://github.com/apache/geode/blob/develop/docker/Dockerfile
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05220.html">[jira] [Assigned] (BEAM-1534) Create a dockerized developer environment for Beam</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BAssigned%5C%5D+%5C%28BEAM%5C-1534%5C%29+Create+a+dockerized+developer+environment+for+Beam%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Nitin+Lamba+%5C%28JIRA%5C%29%22&o=newest&f=1">Nitin Lamba (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1534?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Nitin Lamba reassigned BEAM-1534:
    -
    
    Assignee: Frances Perry  (was: Nitin Lamba)
    
    > Create a dockerized developer environment for Beam
    > --
    >
    > Key: BEAM-1534
    > URL: https://issues.apache.org/jira/browse/BEAM-1534
    > Project: Beam
    >  Issue Type: New Feature
    >  Components: examples-java
    >Reporter: Nitin Lamba
    >Assignee: Frances Perry
    >
    > This will help create a repeatable developer environment setup.
    > Other Apache projects can be used as a reference:
    > https://github.com/apache/ambari/tree/trunk/dev-support/docker
    > https://github.com/apache/geode/blob/develop/docker/Dockerfile
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05219.html">[jira] [Commented] (BEAM-646) Get runners out of the apply()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-646%5C%29+Get+runners+out+of+the+apply%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-646?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886740#comment-15886740
     ] 
    
    ASF GitHub Bot commented on BEAM-646:
    -
    
    GitHub user tgroh opened a pull request:
    
    https://github.com/apache/beam/pull/2119
    
    [BEAM-646] Remove PipelineRunner#apply
    
    All existing Pipeline Runners that use the Java SDK modify Pipeline
    graphs with the Pipeline Surgery APIs. Apply is now superflous.
    
    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/tgroh/beam remove_runner_apply
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2119.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 #2119
    
    
    commit b7ea4c86ae081d198a9ec98dd50ba8cb7902cced
    Author: Thomas Groh 
    Date:   2017-02-24T01:32:01Z
    
    Remove PipelineRunner#apply
    
    All existing Pipeline Runners that use the Java SDK modify Pipeline
    graphs with the Pipeline Surgery APIs. Apply is now superflous.
    
    
    
    
    > Get runners out of the apply()
    > --
    >
    > Key: BEAM-646
    > URL: https://issues.apache.org/jira/browse/BEAM-646
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: beam-model-runner-api, sdk-java-core
    >Reporter: Kenneth Knowles
    >Assignee: Thomas Groh
    >  Labels: backwards-incompatible
    >
    > Right now, the runner intercepts calls to apply() and replaces transforms as 
    > we go. This means that there is no "original" user graph. For portability and 
    > misc architectural benefits, we would like to build the original graph first, 
    > and have the runner override later.
    > Some runners already work in this manner, but we could integrate it more 
    > smoothly, with more validation, via some handy APIs on e.g. the Pipeline 
    > object.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05218.html">[GitHub] beam pull request #2119: [BEAM-646] Remove PipelineRunner#apply</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5BGitHub%5C%5D+beam+pull+request+%232119%5C%3A+%5C%5BBEAM%5C-646%5C%5D+Remove+PipelineRunner%23apply%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22tgroh%22&o=newest&f=1">tgroh</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    GitHub user tgroh opened a pull request:
    
    https://github.com/apache/beam/pull/2119
    
    [BEAM-646] Remove PipelineRunner#apply
    
    All existing Pipeline Runners that use the Java SDK modify Pipeline
    graphs with the Pipeline Surgery APIs. Apply is now superflous.
    
    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/tgroh/beam remove_runner_apply
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2119.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 #2119
    
    
    commit b7ea4c86ae081d198a9ec98dd50ba8cb7902cced
    Author: Thomas Groh 
    Date:   2017-02-24T01:32:01Z
    
    Remove PipelineRunner#apply
    
    All existing Pipeline Runners that use the Java SDK modify Pipeline
    graphs with the Pipeline Surgery APIs. Apply is now superflous.
    
    
    
    
    ---
    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.
    ---
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05217.html">[jira] [Commented] (BEAM-646) Get runners out of the apply()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-646%5C%29+Get+runners+out+of+the+apply%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-646?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886732#comment-15886732
     ] 
    
    ASF GitHub Bot commented on BEAM-646:
    -
    
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/2063
    
    
    > Get runners out of the apply()
    > --
    >
    > Key: BEAM-646
    > URL: https://issues.apache.org/jira/browse/BEAM-646
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: beam-model-runner-api, sdk-java-core
    >Reporter: Kenneth Knowles
    >Assignee: Thomas Groh
    >  Labels: backwards-incompatible
    >
    > Right now, the runner intercepts calls to apply() and replaces transforms as 
    > we go. This means that there is no "original" user graph. For portability and 
    > misc architectural benefits, we would like to build the original graph first, 
    > and have the runner override later.
    > Some runners already work in this manner, but we could integrate it more 
    > smoothly, with more validation, via some handy APIs on e.g. the Pipeline 
    > object.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05216.html">[GitHub] beam pull request #2063: [BEAM-646] Get Apex Runner out of the #apply()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5BGitHub%5C%5D+beam+pull+request+%232063%5C%3A+%5C%5BBEAM%5C-646%5C%5D+Get+Apex+Runner+out+of+the+%23apply%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22asfgit%22&o=newest&f=1">asfgit</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/2063
    
    
    ---
    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.
    ---
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05214.html">[2/2] beam git commit: Get ApexRunner out of the #apply()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B2%5C%2F2%5C%5D+beam+git+commit%5C%3A+Get+ApexRunner+out+of+the+%23apply%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22tgroh%22&o=newest&f=1">tgroh</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Get ApexRunner out of the #apply()
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/47333b0e
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/47333b0e
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/47333b0e
    
    Branch: refs/heads/master
    Commit: 47333b0e7882888ffa41298a142e0ca7de2672a3
    Parents: eeed30b
    Author: Thomas Groh 
    Authored: Mon Feb 27 09:38:10 2017 -0800
    Committer: Thomas Groh 
    Committed: Mon Feb 27 15:16:48 2017 -0800
    
    --
     runners/apex/pom.xml|  5 +
     .../apache/beam/runners/apex/ApexRunner.java| 99 
     .../translation/ApexPipelineTranslator.java |  5 +-
     .../translation/CreateValuesTranslator.java | 11 ++-
     .../core/construction/PrimitiveCreate.java  | 77 +++
     5 files changed, 151 insertions(+), 46 deletions(-)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/47333b0e/runners/apex/pom.xml
    --
    diff --git a/runners/apex/pom.xml b/runners/apex/pom.xml
    index f45e2df..f5fe4bc 100644
    --- a/runners/apex/pom.xml
    +++ b/runners/apex/pom.xml
    @@ -87,6 +87,11 @@
     
     
       org.apache.beam
    +  beam-runners-core-construction-java
    +
    +
    +
    +  org.apache.beam
       beam-runners-core-java
       
     
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/47333b0e/runners/apex/src/main/java/org/apache/beam/runners/apex/ApexRunner.java
    --
    diff --git 
    a/runners/apex/src/main/java/org/apache/beam/runners/apex/ApexRunner.java 
    b/runners/apex/src/main/java/org/apache/beam/runners/apex/ApexRunner.java
    index 1eb5e72..010ede3 100644
    --- a/runners/apex/src/main/java/org/apache/beam/runners/apex/ApexRunner.java
    +++ b/runners/apex/src/main/java/org/apache/beam/runners/apex/ApexRunner.java
    @@ -22,7 +22,7 @@ import com.datatorrent.api.Context.DAGContext;
     import com.datatorrent.api.DAG;
     import com.datatorrent.api.StreamingApplication;
     import com.google.common.base.Throwables;
    -
    +import com.google.common.collect.ImmutableMap;
     import java.io.File;
     import java.io.IOException;
     import java.io.InputStream;
    @@ -31,6 +31,7 @@ import java.net.URISyntaxException;
     import java.util.ArrayList;
     import java.util.Collections;
     import java.util.List;
    +import java.util.Map;
     import java.util.Properties;
     import java.util.concurrent.atomic.AtomicReference;
     import org.apache.apex.api.EmbeddedAppLauncher;
    @@ -38,25 +39,30 @@ import org.apache.apex.api.Launcher;
     import org.apache.apex.api.Launcher.AppHandle;
     import org.apache.apex.api.Launcher.LaunchMode;
     import org.apache.beam.runners.apex.translation.ApexPipelineTranslator;
    +import org.apache.beam.runners.core.construction.PTransformMatchers;
    +import org.apache.beam.runners.core.construction.PrimitiveCreate;
    +import 
    org.apache.beam.runners.core.construction.SingleInputOutputOverrideFactory;
     import org.apache.beam.sdk.Pipeline;
     import org.apache.beam.sdk.coders.Coder;
     import org.apache.beam.sdk.coders.CoderRegistry;
     import org.apache.beam.sdk.coders.ListCoder;
     import org.apache.beam.sdk.options.PipelineOptions;
     import org.apache.beam.sdk.options.PipelineOptionsValidator;
    +import org.apache.beam.sdk.runners.PTransformMatcher;
    +import org.apache.beam.sdk.runners.PTransformOverrideFactory;
     import org.apache.beam.sdk.runners.PipelineRunner;
     import org.apache.beam.sdk.transforms.Combine;
    +import org.apache.beam.sdk.transforms.Combine.GloballyAsSingletonView;
     import org.apache.beam.sdk.transforms.Create;
     import org.apache.beam.sdk.transforms.DoFn;
     import org.apache.beam.sdk.transforms.PTransform;
     import org.apache.beam.sdk.transforms.ParDo;
     import org.apache.beam.sdk.transforms.View;
    +import org.apache.beam.sdk.transforms.View.AsIterable;
    +import org.apache.beam.sdk.transforms.View.AsSingleton;
     import org.apache.beam.sdk.util.PCollectionViews;
    -import org.apache.beam.sdk.util.WindowingStrategy;
     import org.apache.beam.sdk.values.PCollection;
     import org.apache.beam.sdk.values.PCollectionView;
    -import org.apache.beam.sdk.values.PInput;
    -import org.apache.beam.sdk.values.POutput;
     import org.apache.hadoop.conf.Configuration;
     
     /**
    @@ -89,37 +95,27 @@ public class ApexRunner extends 
    PipelineRunner {
     return new ApexRunner(apexPipelineOptions);
       }
     
    -  @Override
    -  public  OutputT apply(
    -  PTransform transform, InputT input) {
    -
    -if (Create.Values.class.equals(transform.getClass())) {
    -  return (OutputT) PCollection
    -  .createPrimitiveOutputInternal(
    -  input.getPipeline(),
    -  WindowingStrategy.globalDefault(),
    -  PCollection.IsBounded.BOUNDED);
    -} else if 
    (Combine.GloballyAsSingletonView.class.equals(transform.getClass())) {
    -  PTransform customTransform = (PTransform)
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05215.html">[1/2] beam git commit: This closes #2063</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B1%5C%2F2%5C%5D+beam+git+commit%5C%3A+This+closes+%232063%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22tgroh%22&o=newest&f=1">tgroh</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Repository: beam
    Updated Branches:
      refs/heads/master eeed30b6e -> c8dfb851c
    
    
    This closes #2063
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/c8dfb851
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/c8dfb851
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/c8dfb851
    
    Branch: refs/heads/master
    Commit: c8dfb851cdcb24a9afb01416598d2881e449f079
    Parents: eeed30b 47333b0
    Author: Thomas Groh 
    Authored: Mon Feb 27 15:16:48 2017 -0800
    Committer: Thomas Groh 
    Committed: Mon Feb 27 15:16:48 2017 -0800
    
    --
     runners/apex/pom.xml|  5 +
     .../apache/beam/runners/apex/ApexRunner.java| 99 
     .../translation/ApexPipelineTranslator.java |  5 +-
     .../translation/CreateValuesTranslator.java | 11 ++-
     .../core/construction/PrimitiveCreate.java  | 77 +++
     5 files changed, 151 insertions(+), 46 deletions(-)
    --
    
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05213.html">[jira] [Commented] (BEAM-1569) HDFSFileSource: Unable to read from filePattern with spaces in path</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1569%5C%29+HDFSFileSource%5C%3A+Unable+to+read+from+filePattern+with+spaces+in+path%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Davor+Bonaci+%5C%28JIRA%5C%29%22&o=newest&f=1">Davor Bonaci (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1569?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886721#comment-15886721
     ] 
    
    Davor Bonaci commented on BEAM-1569:
    
    
    [~jbonofre], [~ravwojdyla], any thoughts here?
    [~michel], are you interested in contributing a fix here perhaps?
    
    > HDFSFileSource: Unable to read from filePattern with spaces in path
    > ---
    >
    > Key: BEAM-1569
    > URL: https://issues.apache.org/jira/browse/BEAM-1569
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-core
    >Reporter: Michael Luckey
    >Assignee: Davor Bonaci
    >
    > After the merge of the changes introduced with 
    > https://issues.apache.org/jira/browse/BEAM-1497 we are unable to read from 
    > files containing spaces in path. We encounter following stack trace
    > {noformat}
    > java.lang.reflect.UndeclaredThrowableException
    >   at 
    > org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1713)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource.validate(HDFSFileSource.java:337)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource.createReader(HDFSFileSource.java:329)
    >   at 
    > org.apache.beam.sdk.testing.SourceTestUtils.readFromSource(SourceTestUtils.java:138)
    > Caused by: java.net.URISyntaxException: Illegal character in path at index 
    > 77: 
    > /var/folders/1t/s9pcmfj50nxbt68h3_2z_5wcgn/T/junit6887354597440386901/tmp 
    > data.seq
    >   at java.net.URI$Parser.fail(URI.java:2848)
    >   at java.net.URI$Parser.checkChars(URI.java:3021)
    >   at java.net.URI$Parser.parseHierarchical(URI.java:3105)
    >   at java.net.URI$Parser.parse(URI.java:3063)
    >   at java.net.URI.(URI.java:588)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:340)
    >   at 
    > org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:337)
    >   at java.security.AccessController.doPrivileged(Native Method)
    >   at javax.security.auth.Subject.doAs(Subject.java:422)
    >   at 
    > org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1698)
    >   ... 40 more
    > {noformat}
    > This can be reproduced for instance by 
    > {noformat}
    >   // shameless copy of existing test case
    >   @Test
    >   public void testFullyReadSingleFileWithSpaces() throws Exception {
    > PipelineOptions options = PipelineOptionsFactory.create();
    > List> expectedResults = createRandomRecords(3, 10, 
    > 0);
    > File file = createFileWithData("tmp data.seq", expectedResults);
    > HDFSFileSource, IntWritable, Text> source =
    > HDFSFileSource.from(
    > file.toString(), SequenceFileInputFormat.class, 
    > IntWritable.class, Text.class);
    > assertEquals(file.length(), source.getEstimatedSizeBytes(null));
    > assertThat(expectedResults, containsInAnyOrder(readFromSource(source, 
    > options).toArray()));
    >   }
    > {noformat}
    > Changing the implementation slightly to
    > {noformat}
    > diff --git 
    > a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    >  
    > b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > index 2a731fb..df72643 100644
    > --- 
    > a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > +++ 
    > b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    > @@ -30,7 +30,6 @@ import java.io.ObjectInput;
    >  import java.io.ObjectOutput;
    >  import java.lang.reflect.InvocationTargetException;
    >  import java.lang.reflect.Method;
    > -import java.net.URI;
    >  import java.security.PrivilegedExceptionAction;
    >  import java.util.List;
    >  import java.util.ListIterator;
    > @@ -337,9 +336,10 @@ public abstract class HDFSFileSource extends 
    > BoundedSource {
    >  UGIHelper.getBestUGI(username()).doAs(new 
    > PrivilegedExceptionAction() {
    >@Override
    >public Void run() throws Exception {
    > -FileSystem fs = FileSystem.get(new URI(filepattern()),
    > +final Path pathPattern = new Path(filepattern());
    > +FileSystem fs = FileSystem.get(pathPattern.toUri(),
    >  
    > SerializableConfiguration.newConfiguration(serializableConfiguration()));
    > -FileStatus[] fileStatuses = fs.globStatus(new 
    > Path(filepattern()));
    > +FileStatus[] fileStatuses = fs.globStatus(pathPattern);
    >  checkState(
    >  fileStatuses != null && fileStatuses.length > 0,
    >  "Unable to find any files matching %s", filepattern());
    > {noformat}
    > seems to be fixing the issue for us.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05212.html">[jira] [Closed] (BEAM-1540) Refactor HBaseIO to hide visibility of Coders/Serializable classes</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BClosed%5C%5D+%5C%28BEAM%5C-1540%5C%29+Refactor+HBaseIO+to+hide+visibility+of+Coders%5C%2FSerializable+classes%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Daniel+Halperin+%5C%28JIRA%5C%29%22&o=newest&f=1">Daniel Halperin (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Daniel Halperin closed BEAM-1540.
    -
    Resolution: Fixed
    
    Removed `Fix Version` as we do not need it for issues unless they were present 
    in a prior release.
    
    > Refactor HBaseIO to hide visibility of Coders/Serializable classes
    > --
    >
    > Key: BEAM-1540
    > URL: https://issues.apache.org/jira/browse/BEAM-1540
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: sdk-java-extensions
    >Reporter: Ismaël Mejía
    >Assignee: Ismaël Mejía
    >Priority: Trivial
    > Fix For: Not applicable
    >
    >
    
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05211.html">[jira] [Updated] (BEAM-1540) Refactor HBaseIO to hide visibility of Coders/Serializable classes</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BUpdated%5C%5D+%5C%28BEAM%5C-1540%5C%29+Refactor+HBaseIO+to+hide+visibility+of+Coders%5C%2FSerializable+classes%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Daniel+Halperin+%5C%28JIRA%5C%29%22&o=newest&f=1">Daniel Halperin (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Daniel Halperin updated BEAM-1540:
    --
    Fix Version/s: (was: 0.6.0)
       Not applicable
    
    > Refactor HBaseIO to hide visibility of Coders/Serializable classes
    > --
    >
    > Key: BEAM-1540
    > URL: https://issues.apache.org/jira/browse/BEAM-1540
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: sdk-java-extensions
    >Reporter: Ismaël Mejía
    >Assignee: Ismaël Mejía
    >Priority: Trivial
    > Fix For: Not applicable
    >
    >
    
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05210.html">[jira] [Reopened] (BEAM-1540) Refactor HBaseIO to hide visibility of Coders/Serializable classes</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BReopened%5C%5D+%5C%28BEAM%5C-1540%5C%29+Refactor+HBaseIO+to+hide+visibility+of+Coders%5C%2FSerializable+classes%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Daniel+Halperin+%5C%28JIRA%5C%29%22&o=newest&f=1">Daniel Halperin (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Daniel Halperin reopened BEAM-1540:
    ---
    
    > Refactor HBaseIO to hide visibility of Coders/Serializable classes
    > --
    >
    > Key: BEAM-1540
    > URL: https://issues.apache.org/jira/browse/BEAM-1540
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: sdk-java-extensions
    >Reporter: Ismaël Mejía
    >Assignee: Ismaël Mejía
    >Priority: Trivial
    > Fix For: Not applicable
    >
    >
    
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05209.html">[jira] [Commented] (BEAM-1160) Disabling Read transform validation cause empty file patterns to unexpected succeed</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1160%5C%29+Disabling+Read+transform+validation+cause+empty+file+patterns+to+unexpected+succeed%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1160?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886697#comment-15886697
     ] 
    
    ASF GitHub Bot commented on BEAM-1160:
    --
    
    Github user swegner closed the pull request at:
    
    https://github.com/apache/beam/pull/1627
    
    
    > Disabling Read transform validation cause empty file patterns to unexpected 
    > succeed
    > ---
    >
    > Key: BEAM-1160
    > URL: https://issues.apache.org/jira/browse/BEAM-1160
    > Project: Beam
    >  Issue Type: Bug
    >  Components: sdk-java-core
    >Reporter: Scott Wegner
    >Assignee: Scott Wegner
    >
    > Typically, input file patterns are validated during Pipeline construction, 
    > but standard Read transforms include an option to disable validation. This is 
    > generally useful but can lead to cases where a Pipeline executes successfully 
    > with empty inputs.
    > We should fail execution on empty file-based inputs even when validation is 
    > disabled.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05208.html">[GitHub] beam pull request #1627: [BEAM-1160] Add option to disable failures if fileP...</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5BGitHub%5C%5D+beam+pull+request+%231627%5C%3A+%5C%5BBEAM%5C-1160%5C%5D+Add+option+to+disable+failures+if+fileP...%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22swegner%22&o=newest&f=1">swegner</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Github user swegner closed the pull request at:
    
    https://github.com/apache/beam/pull/1627
    
    
    ---
    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.
    ---
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05207.html">Build failed in Jenkins: beam_PostCommit_Java_RunnableOnService_Dataflow #2421</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22Build+failed+in+Jenkins%5C%3A+beam_PostCommit_Java_RunnableOnService_Dataflow+%232421%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Apache+Jenkins+Server%22&o=newest&f=1">Apache Jenkins Server</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    See 
    <https://builds.apache.org/job/beam_PostCommit_Java_RunnableOnService_Dataflow/2421/display/redirect?page=changes>
    
    Changes:
    
    [jbonofre] [BEAM-1466] JSON utils extension
    
    [dhalperi] [BEAM-1540] Move coders package classes to the top level
    
    [dhalperi] [BEAM-1540] Restrict access level for Coders/Serializable classes
    
    [dhalperi] [BEAM-1540] Small refactorings for HBaseIO
    
    [dhalperi] [BEAM-1549] Add missing tests for Coders/Serializers in HBaseIO
    
    --
    [...truncated 142.58 KB...]
    2017-02-27T22:29:51.323 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/reporting/maven-reporting-api/3.0/maven-reporting-api-3.0.jar
     (11 KB at 296.8 KB/sec)
    2017-02-27T22:29:51.323 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-repository-metadata/3.0/maven-repository-metadata-3.0.jar
    2017-02-27T22:29:51.326 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/reporting/maven-reporting-exec/1.2/maven-reporting-exec-1.2.jar
     (27 KB at 621.9 KB/sec)
    2017-02-27T22:29:51.326 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-model-builder/3.0/maven-model-builder-3.0.jar
    2017-02-27T22:29:51.331 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/sonatype/aether/aether-util/1.7/aether-util-1.7.jar
     (106 KB at 2510.0 KB/sec)
    2017-02-27T22:29:51.331 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-aether-provider/3.0/maven-aether-provider-3.0.jar
    2017-02-27T22:29:51.340 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/sonatype/aether/aether-impl/1.7/aether-impl-1.7.jar
    2017-02-27T22:29:51.349 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/shared/maven-shared-utils/0.3/maven-shared-utils-0.3.jar
     (152 KB at 2486.5 KB/sec)
    2017-02-27T22:29:51.349 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/sonatype/aether/aether-spi/1.7/aether-spi-1.7.jar
    2017-02-27T22:29:51.362 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-aether-provider/3.0/maven-aether-provider-3.0.jar
     (50 KB at 694.3 KB/sec)
    2017-02-27T22:29:51.362 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/sonatype/aether/aether-api/1.7/aether-api-1.7.jar
    2017-02-27T22:29:51.369 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-model-builder/3.0/maven-model-builder-3.0.jar
     (145 KB at 1829.2 KB/sec)
    2017-02-27T22:29:51.369 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-interpolation/1.14/plexus-interpolation-1.14.jar
    2017-02-27T22:29:51.383 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/sonatype/aether/aether-impl/1.7/aether-impl-1.7.jar
     (104 KB at 1116.0 KB/sec)
    2017-02-27T22:29:51.383 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-settings/3.0/maven-settings-3.0.jar
    2017-02-27T22:29:51.392 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/sonatype/aether/aether-spi/1.7/aether-spi-1.7.jar
     (14 KB at 129.7 KB/sec)
    2017-02-27T22:29:51.392 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-settings-builder/3.0/maven-settings-builder-3.0.jar
    2017-02-27T22:29:51.403 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-interpolation/1.14/plexus-interpolation-1.14.jar
     (60 KB at 527.8 KB/sec)
    2017-02-27T22:29:51.403 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-archiver/2.5/maven-archiver-2.5.jar
    2017-02-27T22:29:51.409 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/sonatype/aether/aether-api/1.7/aether-api-1.7.jar
     (73 KB at 608.7 KB/sec)
    2017-02-27T22:29:51.410 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-archiver/2.9/plexus-archiver-2.9.jar
    2017-02-27T22:29:51.414 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-settings/3.0/maven-settings-3.0.jar
     (46 KB at 367.5 KB/sec)
    2017-02-27T22:29:51.414 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-io/2.4/plexus-io-2.4.jar
    2017-02-27T22:29:51.415 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-repository-metadata/3.0/maven-repository-metadata-3.0.jar
     (30 KB at 235.0 KB/sec)
    2017-02-27T22:29:51.415 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/apache/commons/commons-compress/1.9/commons-compress-1.9.jar
    2017-02-27T22:29:51.426 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-settings-builder/3.0/maven-settings-builder-3.0.jar
     (37 KB at 271.3 KB/sec)
    2017-02-27T22:29:51.426 [INFO] Downloading: 
    https://repo.maven.apache.org/maven2/org/codehaus/plexus/plexus-i18n/1.0-beta-7/plexus-i18n-1.0-beta-7.jar
    2017-02-27T22:29:51.431 [INFO] Downloaded: 
    https://repo.maven.apache.org/maven2/org/apache/maven/maven-archiver/2.5/maven-archiver-2.5.jar
     (22 K</pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05206.html">[jira] [Closed] (BEAM-1540) Refactor HBaseIO to hide visibility of Coders/Serializable classes</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BClosed%5C%5D+%5C%28BEAM%5C-1540%5C%29+Refactor+HBaseIO+to+hide+visibility+of+Coders%5C%2FSerializable+classes%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22JIRA%22&o=newest&f=1">JIRA</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Ismaël Mejía closed BEAM-1540.
    --
    
    > Refactor HBaseIO to hide visibility of Coders/Serializable classes
    > --
    >
    > Key: BEAM-1540
    > URL: https://issues.apache.org/jira/browse/BEAM-1540
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: sdk-java-extensions
    >Reporter: Ismaël Mejía
    >Assignee: Ismaël Mejía
    >Priority: Trivial
    > Fix For: 0.6.0
    >
    >
    
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05205.html">[jira] [Resolved] (BEAM-1540) Refactor HBaseIO to hide visibility of Coders/Serializable classes</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BResolved%5C%5D+%5C%28BEAM%5C-1540%5C%29+Refactor+HBaseIO+to+hide+visibility+of+Coders%5C%2FSerializable+classes%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22JIRA%22&o=newest&f=1">JIRA</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Ismaël Mejía resolved BEAM-1540.
    
       Resolution: Fixed
    Fix Version/s: 0.6.0
    
    > Refactor HBaseIO to hide visibility of Coders/Serializable classes
    > --
    >
    > Key: BEAM-1540
    > URL: https://issues.apache.org/jira/browse/BEAM-1540
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: sdk-java-extensions
    >Reporter: Ismaël Mejía
    >Assignee: Ismaël Mejía
    >Priority: Trivial
    > Fix For: 0.6.0
    >
    >
    
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05204.html">Jenkins build is back to stable : beam_PostCommit_Java_MavenInstall #2764</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22Jenkins+build+is+back+to+stable+%5C%3A+beam_PostCommit_Java_MavenInstall+%232764%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Apache+Jenkins+Server%22&o=newest&f=1">Apache Jenkins Server</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    See 
    <https://builds.apache.org/job/beam_PostCommit_Java_MavenInstall/2764/display/redirect?page=changes>
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05203.html">[jira] [Created] (BEAM-1569) HDFSFileSource: Unable to read from filePattern with spaces in path</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCreated%5C%5D+%5C%28BEAM%5C-1569%5C%29+HDFSFileSource%5C%3A+Unable+to+read+from+filePattern+with+spaces+in+path%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Michael+Luckey+%5C%28JIRA%5C%29%22&o=newest&f=1">Michael Luckey (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Michael Luckey created BEAM-1569:
    
    
     Summary: HDFSFileSource: Unable to read from filePattern with 
    spaces in path
     Key: BEAM-1569
     URL: https://issues.apache.org/jira/browse/BEAM-1569
     Project: Beam
      Issue Type: Bug
      Components: sdk-java-core
    Reporter: Michael Luckey
    Assignee: Davor Bonaci
    
    
    After the merge of the changes introduced with 
    https://issues.apache.org/jira/browse/BEAM-1497 we are unable to read from 
    files containing spaces in path. We encounter following stack trace
    
    {noformat}
    java.lang.reflect.UndeclaredThrowableException
    at 
    org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1713)
    at 
    org.apache.beam.sdk.io.hdfs.HDFSFileSource.validate(HDFSFileSource.java:337)
    at 
    org.apache.beam.sdk.io.hdfs.HDFSFileSource.createReader(HDFSFileSource.java:329)
    at 
    org.apache.beam.sdk.testing.SourceTestUtils.readFromSource(SourceTestUtils.java:138)
    
    Caused by: java.net.URISyntaxException: Illegal character in path at index 77: 
    /var/folders/1t/s9pcmfj50nxbt68h3_2z_5wcgn/T/junit6887354597440386901/tmp 
    data.seq
    at java.net.URI$Parser.fail(URI.java:2848)
    at java.net.URI$Parser.checkChars(URI.java:3021)
    at java.net.URI$Parser.parseHierarchical(URI.java:3105)
    at java.net.URI$Parser.parse(URI.java:3063)
    at java.net.URI.(URI.java:588)
    at 
    org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:340)
    at 
    org.apache.beam.sdk.io.hdfs.HDFSFileSource$7.run(HDFSFileSource.java:337)
    at java.security.AccessController.doPrivileged(Native Method)
    at javax.security.auth.Subject.doAs(Subject.java:422)
    at 
    org.apache.hadoop.security.UserGroupInformation.doAs(UserGroupInformation.java:1698)
    ... 40 more
    {noformat}
    
    
    This can be reproduced for instance by 
    
    {noformat}
      // shameless copy of existing test case
      @Test
      public void testFullyReadSingleFileWithSpaces() throws Exception {
    PipelineOptions options = PipelineOptionsFactory.create();
    List> expectedResults = createRandomRecords(3, 10, 0);
    File file = createFileWithData("tmp data.seq", expectedResults);
    
    HDFSFileSource, IntWritable, Text> source =
    HDFSFileSource.from(
    file.toString(), SequenceFileInputFormat.class, 
    IntWritable.class, Text.class);
    
    assertEquals(file.length(), source.getEstimatedSizeBytes(null));
    
    assertThat(expectedResults, containsInAnyOrder(readFromSource(source, 
    options).toArray()));
      }
    {noformat}
    
    Changing the implementation slightly to
    
    {noformat}
    diff --git 
    a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
     
    b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    index 2a731fb..df72643 100644
    --- 
    a/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    +++ 
    b/sdks/java/io/hdfs/src/main/java/org/apache/beam/sdk/io/hdfs/HDFSFileSource.java
    @@ -30,7 +30,6 @@ import java.io.ObjectInput;
     import java.io.ObjectOutput;
     import java.lang.reflect.InvocationTargetException;
     import java.lang.reflect.Method;
    -import java.net.URI;
     import java.security.PrivilegedExceptionAction;
     import java.util.List;
     import java.util.ListIterator;
    @@ -337,9 +336,10 @@ public abstract class HDFSFileSource extends 
    BoundedSource {
     UGIHelper.getBestUGI(username()).doAs(new 
    PrivilegedExceptionAction() {
       @Override
       public Void run() throws Exception {
    -FileSystem fs = FileSystem.get(new URI(filepattern()),
    +final Path pathPattern = new Path(filepattern());
    +FileSystem fs = FileSystem.get(pathPattern.toUri(),
     
    SerializableConfiguration.newConfiguration(serializableConfiguration()));
    -FileStatus[] fileStatuses = fs.globStatus(new 
    Path(filepattern()));
    +FileStatus[] fileStatuses = fs.globStatus(pathPattern);
     checkState(
     fileStatuses != null && fileStatuses.length > 0,
     "Unable to find any files matching %s", filepattern());
    {noformat}
    seems to be fixing the issue for us.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05202.html">[2/5] beam git commit: [BEAM-1540] Small refactorings for HBaseIO</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B2%5C%2F5%5C%5D+beam+git+commit%5C%3A+%5C%5BBEAM%5C-1540%5C%5D+Small+refactorings+for+HBaseIO%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22dhalperi%22&o=newest&f=1">dhalperi</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    [BEAM-1540] Small refactorings for HBaseIO
    
    Rename result to current + make some attributes final
    Refactor SerializableScan to use get()
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/16a2bc0e
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/16a2bc0e
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/16a2bc0e
    
    Branch: refs/heads/master
    Commit: 16a2bc0ef860b02f5a8da01b0bd808973310c206
    Parents: 715b95a
    Author: Ismaël Mejía 
    Authored: Thu Feb 23 10:23:04 2017 +0100
    Committer: Dan Halperin 
    Committed: Mon Feb 27 13:13:03 2017 -0800
    
    --
     sdks/java/io/hbase/pom.xml  |  5 ++-
     .../org/apache/beam/sdk/io/hbase/HBaseIO.java   | 32 ++--
     .../beam/sdk/io/hbase/SerializableScan.java | 10 --
     .../apache/beam/sdk/io/hbase/HBaseIOTest.java   |  2 +-
     4 files changed, 27 insertions(+), 22 deletions(-)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/16a2bc0e/sdks/java/io/hbase/pom.xml
    --
    diff --git a/sdks/java/io/hbase/pom.xml b/sdks/java/io/hbase/pom.xml
    index f4a06a9..dfcca7a 100644
    --- a/sdks/java/io/hbase/pom.xml
    +++ b/sdks/java/io/hbase/pom.xml
    @@ -196,9 +196,8 @@
     
     
     
    -  commons-lang
    -  commons-lang
    -  2.6
    +  org.apache.commons
    +  commons-lang3
       test
     
     
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/16a2bc0e/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java
    --
    diff --git 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java 
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java
    index 3c49db6..ed191cb 100644
    --- a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java
    +++ b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java
    @@ -217,7 +217,7 @@ public class HBaseIO {
      */
     public Read withFilter(Filter filter) {
     checkNotNull(filter, "filter");
    -return withScan(serializableScan.getScan().setFilter(filter));
    +return withScan(serializableScan.get().setFilter(filter));
     }
     
     /**
    @@ -229,7 +229,7 @@ public class HBaseIO {
     checkNotNull(keyRange, "keyRange");
     byte[] startRow = keyRange.getStartKey().getBytes();
     byte[] stopRow = keyRange.getEndKey().getBytes();
    -return 
    withScan(serializableScan.getScan().setStartRow(startRow).setStopRow(stopRow));
    +return 
    withScan(serializableScan.get().setStartRow(startRow).setStopRow(stopRow));
     }
     
     /**
    @@ -279,7 +279,7 @@ public class HBaseIO {
     builder.add(DisplayData.item("configuration",
     serializableConfiguration.get().toString()));
     builder.add(DisplayData.item("tableId", tableId));
    -builder.addIfNotNull(DisplayData.item("scan", 
    serializableScan.getScan().toString()));
    +builder.addIfNotNull(DisplayData.item("scan", 
    serializableScan.get().toString()));
     }
     
     public String getTableId() {
    @@ -294,18 +294,18 @@ public class HBaseIO {
      * Returns the range of keys that will be read from the table.
      */
     public ByteKeyRange getKeyRange() {
    -byte[] startRow = serializableScan.getScan().getStartRow();
    -byte[] stopRow = serializableScan.getScan().getStopRow();
    +byte[] startRow = serializableScan.get().getStartRow();
    +byte[] stopRow = serializableScan.get().getStopRow();
     return ByteKeyRange.of(ByteKey.copyFrom(startRow), 
    ByteKey.copyFrom(stopRow));
     }
     
    -private SerializableConfiguration serializableConfiguration;
    -private String tableId;
    -private SerializableScan serializableScan;
    +private final SerializableConfiguration serializableConfiguration;
    +private final String tableId;
    +private final SerializableScan serializableScan;
     }
     
     static class HBaseSource extends BoundedSource {
    -private Read read;
    +private final Read read;
     @Nullable private Long estimatedSizeBytes;
     
     HBaseSource(Read read, @Nullable Long estimatedSizeBytes) {
    @@ -318,7 +318,7 @@ public class HBaseIO {
     if (estimatedSizeBytes == null) {
     estimatedSizeBytes = estimateSizeBytes();
     LOG.debug("Estimated size {} bytes for table {} and scan {}", 
    estimatedSizeBytes,
    -read.tableId, read.serializableScan.getScan());
    +read.tableId, read.serializableScan.get());
     }
     return estimatedSizeBytes;
     }
    @@ -360,7 +360,7 @@ public class HBaseIO {
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05200.html">[4/5] beam git commit: [BEAM-1540] Restrict access level for Coders/Serializable classes</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B4%5C%2F5%5C%5D+beam+git+commit%5C%3A+%5C%5BBEAM%5C-1540%5C%5D+Restrict+access+level+for+Coders%5C%2FSerializable+classes%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22dhalperi%22&o=newest&f=1">dhalperi</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    [BEAM-1540] Restrict access level for Coders/Serializable classes
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/715b95ac
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/715b95ac
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/715b95ac
    
    Branch: refs/heads/master
    Commit: 715b95ac584a57751c02c7f930c20b1e264fcaea
    Parents: 0c857c7
    Author: Ismaël Mejía 
    Authored: Thu Feb 23 10:09:34 2017 +0100
    Committer: Dan Halperin 
    Committed: Mon Feb 27 13:13:03 2017 -0800
    
    --
     .../apache/beam/sdk/io/hbase/HBaseMutationCoder.java |  2 +-
     .../apache/beam/sdk/io/hbase/HBaseResultCoder.java   | 15 ---
     .../apache/beam/sdk/io/hbase/SerializableScan.java   |  2 +-
     3 files changed, 10 insertions(+), 9 deletions(-)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/715b95ac/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
    --
    diff --git 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
     
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
    index 356abc4..228e0b4 100644
    --- 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
    +++ 
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
    @@ -35,7 +35,7 @@ import 
    org.apache.hadoop.hbase.protobuf.generated.ClientProtos.MutationProto.Mut
      * A {@link Coder} that serializes and deserializes the {@link Mutation} 
    objects using {@link
      * ProtobufUtil}.
      */
    -public class HBaseMutationCoder extends AtomicCoder implements 
    Serializable {
    +class HBaseMutationCoder extends AtomicCoder implements Serializable 
    {
       private static final HBaseMutationCoder INSTANCE = new HBaseMutationCoder();
     
       private HBaseMutationCoder() {}
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/715b95ac/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseResultCoder.java
    --
    diff --git 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseResultCoder.java
     
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseResultCoder.java
    index 8e5e128..94f324a 100644
    --- 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseResultCoder.java
    +++ 
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseResultCoder.java
    @@ -32,23 +32,24 @@ import 
    org.apache.hadoop.hbase.protobuf.generated.ClientProtos;
      * A {@link Coder} that serializes and deserializes the {@link Result} objects 
    using {@link
      * ProtobufUtil}.
      */
    -public class HBaseResultCoder extends AtomicCoder implements 
    Serializable {
    -
    +class HBaseResultCoder extends AtomicCoder implements Serializable {
       private static final HBaseResultCoder INSTANCE = new HBaseResultCoder();
     
    +  private HBaseResultCoder() {}
    +
       public static HBaseResultCoder of() {
     return INSTANCE;
       }
     
       @Override
    -  public Result decode(InputStream inputStream, Coder.Context context)
    -  throws IOException {
    -return 
    ProtobufUtil.toResult(ClientProtos.Result.parseDelimitedFrom(inputStream));
    +  public void encode(Result value, OutputStream outputStream, Coder.Context 
    context)
    +  throws IOException {
    +ProtobufUtil.toResult(value).writeDelimitedTo(outputStream);
       }
     
       @Override
    -  public void encode(Result value, OutputStream outputStream, Coder.Context 
    context)
    +  public Result decode(InputStream inputStream, Coder.Context context)
       throws IOException {
    -ProtobufUtil.toResult(value).writeDelimitedTo(outputStream);
    +return 
    ProtobufUtil.toResult(ClientProtos.Result.parseDelimitedFrom(inputStream));
       }
     }
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/715b95ac/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/SerializableScan.java
    --
    diff --git 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/SerializableScan.java
     
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/SerializableScan.java
    index ed2ec9e..df575b0 100644
    --- 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/SerializableScan.java
    +++ 
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/SerializableScan.java
    @@ -28,7 +28,7 @@ import 
    org.apache.hadoop.hbase.protobuf.generated.ClientProtos;
     /**
      * This is just a wrapper class to serialize HBase {@link Scan}.
      */
    -public class SerializableScan implements Serializable {
    +class SerializableScan implements Serializable {
     private transient Scan scan;
     
     public SerializableScan(Scan scan) {
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05201.html">[GitHub] beam pull request #2117: [BEAM-1540] Refactor HBaseIO to hide visibility of ...</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5BGitHub%5C%5D+beam+pull+request+%232117%5C%3A+%5C%5BBEAM%5C-1540%5C%5D+Refactor+HBaseIO+to+hide+visibility+of+...%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22asfgit%22&o=newest&f=1">asfgit</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/2117
    
    
    ---
    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.
    ---
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05199.html">[jira] [Commented] (BEAM-1540) Refactor HBaseIO to hide visibility of Coders/Serializable classes</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1540%5C%29+Refactor+HBaseIO+to+hide+visibility+of+Coders%5C%2FSerializable+classes%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886540#comment-15886540
     ] 
    
    ASF GitHub Bot commented on BEAM-1540:
    --
    
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/2117
    
    
    > Refactor HBaseIO to hide visibility of Coders/Serializable classes
    > --
    >
    > Key: BEAM-1540
    > URL: https://issues.apache.org/jira/browse/BEAM-1540
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: sdk-java-extensions
    >Reporter: Ismaël Mejía
    >Assignee: Ismaël Mejía
    >Priority: Trivial
    >
    
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05196.html">[5/5] beam git commit: This closes #2117</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B5%5C%2F5%5C%5D+beam+git+commit%5C%3A+This+closes+%232117%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22dhalperi%22&o=newest&f=1">dhalperi</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    This closes #2117
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/eeed30b6
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/eeed30b6
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/eeed30b6
    
    Branch: refs/heads/master
    Commit: eeed30b6ed2921ed8a57a39a9d10d892995f2f1e
    Parents: 8341924 232975c
    Author: Dan Halperin 
    Authored: Mon Feb 27 13:13:09 2017 -0800
    Committer: Dan Halperin 
    Committed: Mon Feb 27 13:13:09 2017 -0800
    
    --
     sdks/java/io/hbase/pom.xml  |  5 +-
     .../org/apache/beam/sdk/io/hbase/HBaseIO.java   | 35 +-
     .../beam/sdk/io/hbase/HBaseMutationCoder.java   | 71 
     .../beam/sdk/io/hbase/HBaseResultCoder.java | 55 +++
     .../beam/sdk/io/hbase/SerializableScan.java | 55 +++
     .../sdk/io/hbase/coders/HBaseMutationCoder.java | 71 
     .../sdk/io/hbase/coders/HBaseResultCoder.java   | 54 ---
     .../sdk/io/hbase/coders/SerializableScan.java   | 49 --
     .../beam/sdk/io/hbase/coders/package-info.java  | 24 ---
     .../apache/beam/sdk/io/hbase/HBaseIOTest.java   |  2 +-
     .../sdk/io/hbase/HBaseMutationCoderTest.java| 52 ++
     .../beam/sdk/io/hbase/HBaseResultCoderTest.java | 41 +++
     .../beam/sdk/io/hbase/SerializableScanTest.java | 56 +++
     13 files changed, 349 insertions(+), 221 deletions(-)
    --
    
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05197.html">[3/5] beam git commit: [BEAM-1540] Move coders package classes to the top level</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B3%5C%2F5%5C%5D+beam+git+commit%5C%3A+%5C%5BBEAM%5C-1540%5C%5D+Move+coders+package+classes+to+the+top+level%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22dhalperi%22&o=newest&f=1">dhalperi</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    [BEAM-1540] Move coders package classes to the top level
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/0c857c7e
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/0c857c7e
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/0c857c7e
    
    Branch: refs/heads/master
    Commit: 0c857c7e9efe83715111d8f984d09b5d9697448f
    Parents: 8341924
    Author: Ismaël Mejía 
    Authored: Thu Feb 23 09:40:58 2017 +0100
    Committer: Dan Halperin 
    Committed: Mon Feb 27 13:13:03 2017 -0800
    
    --
     .../org/apache/beam/sdk/io/hbase/HBaseIO.java   |  3 -
     .../beam/sdk/io/hbase/HBaseMutationCoder.java   | 71 
     .../beam/sdk/io/hbase/HBaseResultCoder.java | 54 +++
     .../beam/sdk/io/hbase/SerializableScan.java | 49 ++
     .../sdk/io/hbase/coders/HBaseMutationCoder.java | 71 
     .../sdk/io/hbase/coders/HBaseResultCoder.java   | 54 ---
     .../sdk/io/hbase/coders/SerializableScan.java   | 49 --
     .../beam/sdk/io/hbase/coders/package-info.java  | 24 ---
     8 files changed, 174 insertions(+), 201 deletions(-)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/0c857c7e/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java
    --
    diff --git 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java 
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java
    index 75f5615..3c49db6 100644
    --- a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java
    +++ b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseIO.java
    @@ -40,9 +40,6 @@ import org.apache.beam.sdk.coders.IterableCoder;
     import org.apache.beam.sdk.coders.KvCoder;
     import org.apache.beam.sdk.io.BoundedSource;
     import org.apache.beam.sdk.io.hadoop.SerializableConfiguration;
    -import org.apache.beam.sdk.io.hbase.coders.HBaseMutationCoder;
    -import org.apache.beam.sdk.io.hbase.coders.HBaseResultCoder;
    -import org.apache.beam.sdk.io.hbase.coders.SerializableScan;
     import org.apache.beam.sdk.io.range.ByteKey;
     import org.apache.beam.sdk.io.range.ByteKeyRange;
     import org.apache.beam.sdk.options.PipelineOptions;
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/0c857c7e/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
    --
    diff --git 
    a/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
     
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
    new file mode 100644
    index 000..356abc4
    --- /dev/null
    +++ 
    b/sdks/java/io/hbase/src/main/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoder.java
    @@ -0,0 +1,71 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + *
    + * http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +package org.apache.beam.sdk.io.hbase;
    +
    +import java.io.IOException;
    +import java.io.InputStream;
    +import java.io.OutputStream;
    +import java.io.Serializable;
    +
    +import org.apache.beam.sdk.coders.AtomicCoder;
    +import org.apache.beam.sdk.coders.Coder;
    +import org.apache.hadoop.hbase.client.Delete;
    +import org.apache.hadoop.hbase.client.Mutation;
    +import org.apache.hadoop.hbase.client.Put;
    +import org.apache.hadoop.hbase.protobuf.ProtobufUtil;
    +import org.apache.hadoop.hbase.protobuf.generated.ClientProtos.MutationProto;
    +import 
    org.apache.hadoop.hbase.protobuf.generated.ClientProtos.MutationProto.MutationType;
    +
    +/**
    + * A {@link Coder} that serializes and deserializes the {@link Mutation} 
    objects using {@link
    + * ProtobufUtil}.
    + */
    +public class HBaseMutationCoder extends AtomicCoder implements 
    Serializable {
    +  private static final HBaseMutationCoder INSTANCE = new HBaseMutationCoder();
    +
    +  private HBaseMutationCoder() {}
    +
    +  public static HBaseMutationCoder of() {
    +return INSTANCE;
    +  }
    +
    +  @Override
    +  public void encode(Mutation mutation, OutputStream outStream,
    + Coder.Context context) throws IOException {
    +MutationType type = getType(mut</pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05198.html">[1/5] beam git commit: [BEAM-1549] Add missing tests for Coders/Serializers in HBaseIO</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B1%5C%2F5%5C%5D+beam+git+commit%5C%3A+%5C%5BBEAM%5C-1549%5C%5D+Add+missing+tests+for+Coders%5C%2FSerializers+in+HBaseIO%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22dhalperi%22&o=newest&f=1">dhalperi</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Repository: beam
    Updated Branches:
      refs/heads/master 83419241a -> eeed30b6e
    
    
    [BEAM-1549] Add missing tests for Coders/Serializers in HBaseIO
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/232975c6
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/232975c6
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/232975c6
    
    Branch: refs/heads/master
    Commit: 232975c6082210f91b6d2542e9afffc04d350d60
    Parents: 16a2bc0
    Author: Ismaël Mejía 
    Authored: Thu Feb 23 23:15:23 2017 +0100
    Committer: Dan Halperin 
    Committed: Mon Feb 27 13:13:03 2017 -0800
    
    --
     .../sdk/io/hbase/HBaseMutationCoderTest.java| 52 ++
     .../beam/sdk/io/hbase/HBaseResultCoderTest.java | 41 ++
     .../beam/sdk/io/hbase/SerializableScanTest.java | 56 
     3 files changed, 149 insertions(+)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/232975c6/sdks/java/io/hbase/src/test/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoderTest.java
    --
    diff --git 
    a/sdks/java/io/hbase/src/test/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoderTest.java
     
    b/sdks/java/io/hbase/src/test/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoderTest.java
    new file mode 100644
    index 000..5bf2d80
    --- /dev/null
    +++ 
    b/sdks/java/io/hbase/src/test/java/org/apache/beam/sdk/io/hbase/HBaseMutationCoderTest.java
    @@ -0,0 +1,52 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + *
    + * http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +package org.apache.beam.sdk.io.hbase;
    +
    +import org.apache.beam.sdk.testing.CoderProperties;
    +import org.apache.hadoop.hbase.client.Delete;
    +import org.apache.hadoop.hbase.client.Increment;
    +import org.apache.hadoop.hbase.client.Mutation;
    +import org.apache.hadoop.hbase.client.Put;
    +import org.junit.Rule;
    +import org.junit.Test;
    +import org.junit.rules.ExpectedException;
    +import org.junit.runner.RunWith;
    +import org.junit.runners.JUnit4;
    +
    +/**
    + * Tests for HBaseMutationCoder.
    + */
    +@RunWith(JUnit4.class)
    +public class HBaseMutationCoderTest {
    +  @Rule public final ExpectedException thrown = ExpectedException.none();
    +  private static final HBaseMutationCoder CODER = HBaseMutationCoder.of();
    +
    +  @Test
    +  public void testMutationEncoding() throws Exception {
    +Mutation put = new Put("1".getBytes());
    +CoderProperties.structuralValueDecodeEncodeEqual(CODER, put);
    +
    +Mutation delete = new Delete("1".getBytes());
    +CoderProperties.structuralValueDecodeEncodeEqual(CODER, delete);
    +
    +Mutation increment = new Increment("1".getBytes());
    +thrown.expect(IllegalArgumentException.class);
    +thrown.expectMessage("Only Put and Delete are supported");
    +CoderProperties.coderDecodeEncodeEqual(CODER, increment);
    +  }
    +}
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/232975c6/sdks/java/io/hbase/src/test/java/org/apache/beam/sdk/io/hbase/HBaseResultCoderTest.java
    --
    diff --git 
    a/sdks/java/io/hbase/src/test/java/org/apache/beam/sdk/io/hbase/HBaseResultCoderTest.java
     
    b/sdks/java/io/hbase/src/test/java/org/apache/beam/sdk/io/hbase/HBaseResultCoderTest.java
    new file mode 100644
    index 000..c6b27d6
    --- /dev/null
    +++ 
    b/sdks/java/io/hbase/src/test/java/org/apache/beam/sdk/io/hbase/HBaseResultCoderTest.java
    @@ -0,0 +1,41 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See the NOTICE file
    + * distributed with this work for additional information
    + * regarding copyright ownership.  The ASF licenses this file
    + * to you under the Apache License, Version 2.0 (the
    + * "License"); you may not use this file except in compliance
    + * with the License.  You may obtain a copy of the License at
    + *
    + * http://www.apache.org/licenses/LICENSE-2.0
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License</pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05194.html">[jira] [Resolved] (BEAM-1466) JSON utils extension</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BResolved%5C%5D+%5C%28BEAM%5C-1466%5C%29+JSON+utils+extension%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22JIRA%22&o=newest&f=1">JIRA</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Jean-Baptiste Onofré resolved BEAM-1466.
    
       Resolution: Fixed
    Fix Version/s: 0.6.0
    
    > JSON utils extension
    > 
    >
    > Key: BEAM-1466
    > URL: https://issues.apache.org/jira/browse/BEAM-1466
    > Project: Beam
    >  Issue Type: New Feature
    >  Components: sdk-java-extensions
    >Reporter: Aviem Zur
    >Assignee: Aviem Zur
    > Fix For: 0.6.0
    >
    >
    > Create a JSON extension module which will contain transforms to aid with 
    > handling JSONs.
    > Suggested transforms:
    > * Parse JSON strings to type OutputT.
    > * Parse InputT to JSON strings.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05195.html">[jira] [Commented] (BEAM-1466) JSON utils extension</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1466%5C%29+JSON+utils+extension%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1466?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886532#comment-15886532
     ] 
    
    ASF GitHub Bot commented on BEAM-1466:
    --
    
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/1983
    
    
    > JSON utils extension
    > 
    >
    > Key: BEAM-1466
    > URL: https://issues.apache.org/jira/browse/BEAM-1466
    > Project: Beam
    >  Issue Type: New Feature
    >  Components: sdk-java-extensions
    >Reporter: Aviem Zur
    >Assignee: Aviem Zur
    > Fix For: 0.6.0
    >
    >
    > Create a JSON extension module which will contain transforms to aid with 
    > handling JSONs.
    > Suggested transforms:
    > * Parse JSON strings to type OutputT.
    > * Parse InputT to JSON strings.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05193.html">[1/2] beam git commit: [BEAM-1466] JSON utils extension</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B1%5C%2F2%5C%5D+beam+git+commit%5C%3A+%5C%5BBEAM%5C-1466%5C%5D+JSON+utils+extension%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22jbonofre%22&o=newest&f=1">jbonofre</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Repository: beam
    Updated Branches:
      refs/heads/master ae6860db5 -> 83419241a
    
    
    [BEAM-1466] JSON utils extension
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/40bc64c9
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/40bc64c9
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/40bc64c9
    
    Branch: refs/heads/master
    Commit: 40bc64c9b2e4c43ec2cd643123d0954fda3f7cf0
    Parents: ae6860d
    Author: Aviem Zur 
    Authored: Sat Feb 11 17:06:45 2017 +0200
    Committer: Jean-Baptiste Onofré 
    Committed: Mon Feb 27 21:10:41 2017 +0100
    
    --
     sdks/java/extensions/jackson/pom.xml| 124 ++
     .../beam/sdk/extensions/jackson/AsJsons.java|  76 ++
     .../beam/sdk/extensions/jackson/ParseJsons.java |  75 ++
     .../sdk/extensions/jackson/package-info.java|  22 ++
     .../jackson/JacksonTransformsTest.java  | 242 +++
     sdks/java/extensions/pom.xml|   1 +
     6 files changed, 540 insertions(+)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/40bc64c9/sdks/java/extensions/jackson/pom.xml
    --
    diff --git a/sdks/java/extensions/jackson/pom.xml 
    b/sdks/java/extensions/jackson/pom.xml
    new file mode 100644
    index 000..be5c953
    --- /dev/null
    +++ b/sdks/java/extensions/jackson/pom.xml
    @@ -0,0 +1,124 @@
    +
    +
    +http://maven.apache.org/POM/4.0.0";
    + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
    + xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 
    http://maven.apache.org/xsd/maven-4.0.0.xsd";>
    +
    +4.0.0
    +
    +
    +org.apache.beam
    +beam-sdks-java-extensions-parent
    +0.6.0-SNAPSHOT
    +../pom.xml
    +
    +
    +beam-sdks-java-extensions-json-jackson
    +Apache Beam :: SDKs :: Java :: Extensions :: Jackson
    +
    +Jackson extension provides PTransforms for deserializing and 
    generating JSON strings.
    +
    +
    +
    +
    +
    +
    +org.apache.maven.plugins
    +maven-shade-plugin
    +
    +
    +package
    +
    +shade
    +
    +
    +
    +
    +
    com.google.guava:guava
    +
    +
    +
    +
    +com.google.common
    +
    org.apache.beam.sdk.extensions.jackson.repackaged.com.google.common
    +
    +
    +
    com.google.thirdparty
    +
    org.apache.beam.sdk.extensions.jackson.repackaged.com.google.thirdparty
    +
    +
    +
    +
    +
    +
    +
    +
    +
    +
    +org.apache.maven.plugins
    +maven-compiler-plugin
    +
    +
    +org.apache.maven.plugins
    +maven-jar-plugin
    +
    +
    +org.apache.maven.plugins
    +maven-shade-plugin
    +
    +
    +
    +
    +
    +
    +org.apache.beam
    +beam-sdks-java-core
    +
    +
    +
    +com.fasterxml.jackson.core
    +jackson-databind
    +
    +
    +
    +com.google.guava
    +guava
    +
    +
    +
    +
    +
    +org.apache.beam
    +beam-runners-direct-java
    +test
    +
    +
    +
    +org.hamcrest
    +hamcrest-all
    +test
    +
    +
    +
    +junit
    +junit
    +test
    +
    +
    +
    +
    \ No newline at end of file
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/40bc64c9/sdks/java/extensions/jackson/src/main/java/org/apache/beam/sdk/extensions/jackson/AsJsons.java
    --
    diff --git 
    a/sdks/java/extensions/jackson/src/main/java/org/apache/beam/sdk/extensions/jackson/AsJsons.java
     
    b/sdks/java/extensions/jackson/src/main/java/org/apache/beam/sdk/extensions/jackson/AsJsons.java
    new file mode 100644
    index 000..a9c7a9f
    --- /dev/null
    +++ 
    b/sdks/java/extensions/jackson/src/main/java/org/apache/beam/sdk/extensions/jackson/AsJsons.java
    @@ -0,0 +1,76 @@
    +/*
    + * Licensed to the Apache Software Foundation (ASF) under one
    + * or more contributor license agreements.  See </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05192.html">[GitHub] beam pull request #1983: [BEAM-1466] JSON utils extension</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5BGitHub%5C%5D+beam+pull+request+%231983%5C%3A+%5C%5BBEAM%5C-1466%5C%5D+JSON+utils+extension%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22asfgit%22&o=newest&f=1">asfgit</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/1983
    
    
    ---
    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.
    ---
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05191.html">[2/2] beam git commit: [BEAM-1466] This closes #1983</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B2%5C%2F2%5C%5D+beam+git+commit%5C%3A+%5C%5BBEAM%5C-1466%5C%5D+This+closes+%231983%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22jbonofre%22&o=newest&f=1">jbonofre</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    [BEAM-1466] This closes #1983
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/83419241
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/83419241
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/83419241
    
    Branch: refs/heads/master
    Commit: 83419241ad72fa8d1e4d214ff8b728b9bd3163af
    Parents: ae6860d 40bc64c
    Author: Jean-Baptiste Onofré 
    Authored: Mon Feb 27 22:07:53 2017 +0100
    Committer: Jean-Baptiste Onofré 
    Committed: Mon Feb 27 22:07:53 2017 +0100
    
    --
     sdks/java/extensions/jackson/pom.xml| 124 ++
     .../beam/sdk/extensions/jackson/AsJsons.java|  76 ++
     .../beam/sdk/extensions/jackson/ParseJsons.java |  75 ++
     .../sdk/extensions/jackson/package-info.java|  22 ++
     .../jackson/JacksonTransformsTest.java  | 242 +++
     sdks/java/extensions/pom.xml|   1 +
     6 files changed, 540 insertions(+)
    --
    
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05190.html">Jenkins build is back to normal : beam_PostCommit_Java_RunnableOnService_Flink #1775</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22Jenkins+build+is+back+to+normal+%5C%3A+beam_PostCommit_Java_RunnableOnService_Flink+%231775%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Apache+Jenkins+Server%22&o=newest&f=1">Apache Jenkins Server</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    See 
    <https://builds.apache.org/job/beam_PostCommit_Java_RunnableOnService_Flink/1775/display/redirect?page=changes>
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05189.html">Jenkins build is still unstable: beam_PostCommit_Java_MavenInstall #2763</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22Jenkins+build+is+still+unstable%5C%3A+beam_PostCommit_Java_MavenInstall+%232763%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Apache+Jenkins+Server%22&o=newest&f=1">Apache Jenkins Server</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    See 
    <https://builds.apache.org/job/beam_PostCommit_Java_MavenInstall/2763/display/redirect?page=changes>
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05188.html">[jira] [Assigned] (BEAM-1567) hashStream should be closed in PackageUtil#createPackageAttributes()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BAssigned%5C%5D+%5C%28BEAM%5C-1567%5C%29+hashStream+should+be+closed+in+PackageUtil%23createPackageAttributes%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Daniel+Halperin+%5C%28JIRA%5C%29%22&o=newest&f=1">Daniel Halperin (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1567?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Daniel Halperin reassigned BEAM-1567:
    -
    
    Assignee: (was: Davor Bonaci)
    
    > hashStream should be closed in PackageUtil#createPackageAttributes()
    > 
    >
    > Key: BEAM-1567
    > URL: https://issues.apache.org/jira/browse/BEAM-1567
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-dataflow
    >Reporter: Ted Yu
    >Priority: Minor
    >
    > Here is related code:
    > {code}
    >   OutputStream hashStream = Funnels.asOutputStream(hasher);
    > {code}
    > hashStream should be closed upon return from the method
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05187.html">[jira] [Commented] (BEAM-1567) hashStream should be closed in PackageUtil#createPackageAttributes()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1567%5C%29+hashStream+should+be+closed+in+PackageUtil%23createPackageAttributes%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Daniel+Halperin+%5C%28JIRA%5C%29%22&o=newest&f=1">Daniel Halperin (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886492#comment-15886492
     ] 
    
    Daniel Halperin commented on BEAM-1567:
    ---
    
    Actually it was auto-assigned to him by default. :) So if you're interested, 
    would be appreciated.
    
    > hashStream should be closed in PackageUtil#createPackageAttributes()
    > 
    >
    > Key: BEAM-1567
    > URL: https://issues.apache.org/jira/browse/BEAM-1567
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-dataflow
    >Reporter: Ted Yu
    >Priority: Minor
    >
    > Here is related code:
    > {code}
    >   OutputStream hashStream = Funnels.asOutputStream(hasher);
    > {code}
    > hashStream should be closed upon return from the method
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05186.html">[jira] [Commented] (BEAM-111) Use SDK implementation of WritableCoder</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-111%5C%29+Use+SDK+implementation+of+WritableCoder%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-111?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886485#comment-15886485
     ] 
    
    ASF GitHub Bot commented on BEAM-111:
    -
    
    GitHub user iemejia opened a pull request:
    
    https://github.com/apache/beam/pull/2118
    
    [BEAM-111] Move WritableCoder to hadoop-common
    
    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/iemejia/beam BEAM-111-move-writablecoder
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2118.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 #2118
    
    
    commit 14dfc21567464fbc62a332b97f67bad928d50a18
    Author: Ismaël Mejía 
    Date:   2017-02-25T04:20:58Z
    
    [BEAM-111] Move WritableCoder to hadoop-common
    
    
    
    
    > Use SDK implementation of WritableCoder
    > ---
    >
    > Key: BEAM-111
    > URL: https://issues.apache.org/jira/browse/BEAM-111
    > Project: Beam
    >  Issue Type: Improvement
    >  Components: runner-spark
    >Reporter: Amit Sela
    >Assignee: Ismaël Mejía
    >  Labels: easy, starter
    >
    > The Spark runner currently uses it's own implementation of WritableCoder, 
    > should use the one in {{io-hdfs}}.
    > Remove {{org.apache.beam.runners.spark.coders.WritableCoder}}.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05185.html">[GitHub] beam pull request #2118: [BEAM-111] Move WritableCoder to hadoop-common</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5BGitHub%5C%5D+beam+pull+request+%232118%5C%3A+%5C%5BBEAM%5C-111%5C%5D+Move+WritableCoder+to+hadoop%5C-common%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22iemejia%22&o=newest&f=1">iemejia</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    GitHub user iemejia opened a pull request:
    
    https://github.com/apache/beam/pull/2118
    
    [BEAM-111] Move WritableCoder to hadoop-common
    
    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/iemejia/beam BEAM-111-move-writablecoder
    
    Alternatively you can review and apply these changes as the patch at:
    
    https://github.com/apache/beam/pull/2118.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 #2118
    
    
    commit 14dfc21567464fbc62a332b97f67bad928d50a18
    Author: Ismaël Mejía 
    Date:   2017-02-25T04:20:58Z
    
    [BEAM-111] Move WritableCoder to hadoop-common
    
    
    
    
    ---
    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.
    ---
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05184.html">[jira] [Commented] (BEAM-1567) hashStream should be closed in PackageUtil#createPackageAttributes()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1567%5C%29+hashStream+should+be+closed+in+PackageUtil%23createPackageAttributes%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Ted+Yu+%5C%28JIRA%5C%29%22&o=newest&f=1">Ted Yu (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886471#comment-15886471
     ] 
    
    Ted Yu commented on BEAM-1567:
    --
    
    I was looking at code under google-cloud-dataflow-java and happened to see this 
    code.
    
    Looks like Davor has taken this JIRA :-)
    
    > hashStream should be closed in PackageUtil#createPackageAttributes()
    > 
    >
    > Key: BEAM-1567
    > URL: https://issues.apache.org/jira/browse/BEAM-1567
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-dataflow
    >Reporter: Ted Yu
    >Assignee: Davor Bonaci
    >Priority: Minor
    >
    > Here is related code:
    > {code}
    >   OutputStream hashStream = Funnels.asOutputStream(hasher);
    > {code}
    > hashStream should be closed upon return from the method
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05183.html">[jira] [Created] (BEAM-1568) Ineffective null check in IsmFormat#structuralValue</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCreated%5C%5D+%5C%28BEAM%5C-1568%5C%29+Ineffective+null+check+in+IsmFormat%23structuralValue%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Ted+Yu+%5C%28JIRA%5C%29%22&o=newest&f=1">Ted Yu (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Ted Yu created BEAM-1568:
    
    
     Summary: Ineffective null check in IsmFormat#structuralValue
     Key: BEAM-1568
     URL: https://issues.apache.org/jira/browse/BEAM-1568
     Project: Beam
      Issue Type: Bug
      Components: runner-dataflow
    Reporter: Ted Yu
    Assignee: Davor Bonaci
    Priority: Minor
    
    
    {code}
    public Object structuralValue(IsmRecord record) throws Exception {
      checkState(record.getKeyComponents().size() == keyComponentCoders.size(),
      "Expected the number of key component coders %s "
      + "to match the number of key components %s.",
      keyComponentCoders.size(), record.getKeyComponents());
    
      if (record != null && consistentWithEquals()) {
    {code}
    record is de-referenced before the null check.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05182.html">[jira] [Commented] (BEAM-1567) hashStream should be closed in PackageUtil#createPackageAttributes()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1567%5C%29+hashStream+should+be+closed+in+PackageUtil%23createPackageAttributes%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Daniel+Halperin+%5C%28JIRA%5C%29%22&o=newest&f=1">Daniel Halperin (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1567?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886456#comment-15886456
     ] 
    
    Daniel Halperin commented on BEAM-1567:
    ---
    
    Thanks [~te...@apache.org],
    
    This indeed looks like it could be an issue. We may be getting lucky that none 
    of the intermediate structures buffer data today, but this should be fixed.
    
    * Can I ask how you found this?
    * Would you be able to send a PR to fix?
    
    Thanks!
    
    > hashStream should be closed in PackageUtil#createPackageAttributes()
    > 
    >
    > Key: BEAM-1567
    > URL: https://issues.apache.org/jira/browse/BEAM-1567
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-dataflow
    >Reporter: Ted Yu
    >Assignee: Davor Bonaci
    >Priority: Minor
    >
    > Here is related code:
    > {code}
    >   OutputStream hashStream = Funnels.asOutputStream(hasher);
    > {code}
    > hashStream should be closed upon return from the method
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05181.html">Build failed in Jenkins: beam_PostCommit_Java_RunnableOnService_Flink #1774</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22Build+failed+in+Jenkins%5C%3A+beam_PostCommit_Java_RunnableOnService_Flink+%231774%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Apache+Jenkins+Server%22&o=newest&f=1">Apache Jenkins Server</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    See 
    <https://builds.apache.org/job/beam_PostCommit_Java_RunnableOnService_Flink/1774/display/redirect?page=changes>
    
    Changes:
    
    [tgroh] Inject Sharding Strategy in the Direct Runner
    
    --
    [...truncated 562.98 KB...]
    Caused by: java.lang.ClassNotFoundException: 
    org.apache.commons.compress.compressors.deflate.DeflateCompressorOutputStream
    at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
    at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
    at sun.misc.Launcher$AppClassLoader.loadClass(Launcher.java:331)
    at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
    ... 42 more
    
    Running org.apache.beam.runners.flink.FlinkRunnerRegistrarTest
    Tests run: 2, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.001 sec - in 
    org.apache.beam.runners.flink.FlinkRunnerRegistrarTest
    Running org.apache.beam.runners.flink.EncodedValueComparatorTest
    Tests run: 11, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.011 sec - in 
    org.apache.beam.runners.flink.EncodedValueComparatorTest
    
    Results :
    
    Tests in error: 
      GroupByNullKeyTest>StreamingProgramTestBase.testJob:85->testProgram:121 » 
    NoClassDefFound
    
    Tests run: 83, Failures: 0, Errors: 1, Skipped: 0
    
    2017-02-27T20:10:15.146 [ERROR] There are test failures.
    
    Please refer to 
    <https://builds.apache.org/job/beam_PostCommit_Java_RunnableOnService_Flink/ws/runners/flink/runner/target/surefire-reports>
     for the individual test results.
    [JENKINS] Recording test results
    2017-02-27T20:10:16.356 [INFO] 
    2017-02-27T20:10:16.356 [INFO] --- maven-jar-plugin:3.0.2:jar (default-jar) @ 
    beam-runners-flink_2.10 ---
    2017-02-27T20:10:16.394 [INFO] Building jar: 
    <https://builds.apache.org/job/beam_PostCommit_Java_RunnableOnService_Flink/ws/runners/flink/runner/target/beam-runners-flink_2.10-0.6.0-SNAPSHOT.jar>
    2017-02-27T20:10:16.512 [INFO] 
    2017-02-27T20:10:16.512 [INFO] --- maven-site-plugin:3.5.1:attach-descriptor 
    (attach-descriptor) @ beam-runners-flink_2.10 ---
    2017-02-27T20:10:16.614 [INFO] 
    2017-02-27T20:10:16.614 [INFO] --- maven-jar-plugin:3.0.2:test-jar 
    (default-test-jar) @ beam-runners-flink_2.10 ---
    2017-02-27T20:10:16.629 [INFO] Building jar: 
    <https://builds.apache.org/job/beam_PostCommit_Java_RunnableOnService_Flink/ws/runners/flink/runner/target/beam-runners-flink_2.10-0.6.0-SNAPSHOT-tests.jar>
    2017-02-27T20:10:16.713 [INFO] 
    2017-02-27T20:10:16.713 [INFO] --- 
    maven-failsafe-plugin:2.19.1:integration-test (default) @ 
    beam-runners-flink_2.10 ---
    2017-02-27T20:10:16.818 [INFO] Tests are skipped.
    [JENKINS] Recording test results2017-02-27T20:10:16.870 [INFO] 
    2017-02-27T20:10:16.870 [INFO] --- maven-surefire-plugin:2.19.1:test 
    (runnable-on-service-tests) @ beam-runners-flink_2.10 ---
    
    2017-02-27T20:10:16.925 [INFO] Surefire report directory: 
    <https://builds.apache.org/job/beam_PostCommit_Java_RunnableOnService_Flink/ws/runners/flink/runner/target/surefire-reports>
    2017-02-27T20:10:16.926 [INFO] Using configured provider 
    org.apache.maven.surefire.junitcore.JUnitCoreProvider
    2017-02-27T20:10:16.926 [INFO] parallel='none', perCoreThreadCount=true, 
    threadCount=4, useUnlimitedThreads=false, threadCountSuites=0, 
    threadCountClasses=0, threadCountMethods=0, parallelOptimized=true
    
    ---
     T E S T S
    ---
    
    Results :
    
    Tests run: 0, Failures: 0, Errors: 0, Skipped: 0
    
    [JENKINS] Recording test results
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    [JENKINS] Archiving disabled
    2017-02-27T20:10:23.537 [INFO] 
    
    2017-02-27T20:10:23.537 [INFO] Reactor Summary:
    2017-02-27T20:10:23.537 [INFO] 
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: Parent 
    .. SUCCESS [ 17.968 s]
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: SDKs :: Java :: Build Tools 
    . SUCCESS [  5.956 s]
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: SDKs 
     SUCCESS [  3.584 s]
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: SDKs :: Common 
    .. SUCCESS [  1.115 s]
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: SDKs :: Common :: Fn API 
     SUCCESS [ 12.616 s]
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: SDKs :: Common :: Runner API 
     SUCCESS [  4.844 s]
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: SDKs :: Java 
     SUCCESS [  0.923 s]
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: SDKs :: Java :: Core 
     SUCCESS [03:11 min]
    2017-02-27T20:10:23.537 [INFO] Apache Beam :: Runners 
    . SUC</pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05180.html">[jira] [Created] (BEAM-1567) hashStream should be closed in PackageUtil#createPackageAttributes()</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCreated%5C%5D+%5C%28BEAM%5C-1567%5C%29+hashStream+should+be+closed+in+PackageUtil%23createPackageAttributes%5C%28%5C%29%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Ted+Yu+%5C%28JIRA%5C%29%22&o=newest&f=1">Ted Yu (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Ted Yu created BEAM-1567:
    
    
     Summary: hashStream should be closed in 
    PackageUtil#createPackageAttributes()
     Key: BEAM-1567
     URL: https://issues.apache.org/jira/browse/BEAM-1567
     Project: Beam
      Issue Type: Bug
      Components: runner-dataflow
    Reporter: Ted Yu
    Assignee: Davor Bonaci
    Priority: Minor
    
    
    Here is related code:
    {code}
      OutputStream hashStream = Funnels.asOutputStream(hasher);
    {code}
    hashStream should be closed upon return from the method
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05179.html">[jira] [Closed] (BEAM-1566) Flink doesn't support DEFLATE compression</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BClosed%5C%5D+%5C%28BEAM%5C-1566%5C%29+Flink+doesn%27t+support+DEFLATE+compression%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22Daniel+Halperin+%5C%28JIRA%5C%29%22&o=newest&f=1">Daniel Halperin (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
     [ 
    https://issues.apache.org/jira/browse/BEAM-1566?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
     ]
    
    Daniel Halperin closed BEAM-1566.
    -
       Resolution: Fixed
    Fix Version/s: Not applicable
    
    > Flink doesn't support DEFLATE compression
    > -
    >
    > Key: BEAM-1566
    > URL: https://issues.apache.org/jira/browse/BEAM-1566
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-flink
    >Reporter: Daniel Halperin
    >Assignee: Daniel Halperin
    > Fix For: Not applicable
    >
    >
    > Adding support to FileBasedSink/Source for DEFLATE compression broke Flink 
    > because Flink overrides Apache Commons Compression down from 1.9 to 1.4.1.
    > I think the upgrade is safe, because Commons is general well behaved.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05177.html">[GitHub] beam pull request #2116: [BEAM-1566] Flink: upgrade apache commons</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5BGitHub%5C%5D+beam+pull+request+%232116%5C%3A+%5C%5BBEAM%5C-1566%5C%5D+Flink%5C%3A+upgrade+apache+commons%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22asfgit%22&o=newest&f=1">asfgit</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/2116
    
    
    ---
    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.
    ---
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05178.html">[1/2] beam git commit: Flink: upgrade apache commons</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B1%5C%2F2%5C%5D+beam+git+commit%5C%3A+Flink%5C%3A+upgrade+apache+commons%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22dhalperi%22&o=newest&f=1">dhalperi</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    Repository: beam
    Updated Branches:
      refs/heads/master 932bb823f -> ae6860db5
    
    
    Flink: upgrade apache commons
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/223aaac0
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/223aaac0
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/223aaac0
    
    Branch: refs/heads/master
    Commit: 223aaac0e0b25dc55284ffcb0908918b61ff7d57
    Parents: 932bb82
    Author: Dan Halperin 
    Authored: Mon Feb 27 11:03:32 2017 -0800
    Committer: Dan Halperin 
    Committed: Mon Feb 27 12:03:48 2017 -0800
    
    --
     runners/flink/runner/pom.xml | 9 +
     1 file changed, 9 insertions(+)
    --
    
    
    http://git-wip-us.apache.org/repos/asf/beam/blob/223aaac0/runners/flink/runner/pom.xml
    --
    diff --git a/runners/flink/runner/pom.xml b/runners/flink/runner/pom.xml
    index d821ca0..c00b328 100644
    --- a/runners/flink/runner/pom.xml
    +++ b/runners/flink/runner/pom.xml
    @@ -186,6 +186,15 @@
       
     
     
    +
    +
    +  org.apache.commons
    +  commons-compress
    +  [1.9,)
    +
    +
     
     
     
    
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05176.html">[jira] [Commented] (BEAM-1566) Flink doesn't support DEFLATE compression</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5Bjira%5C%5D+%5C%5BCommented%5C%5D+%5C%28BEAM%5C-1566%5C%29+Flink+doesn%27t+support+DEFLATE+compression%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22ASF+GitHub+Bot+%5C%28JIRA%5C%29%22&o=newest&f=1">ASF GitHub Bot (JIRA)</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    
    [ 
    https://issues.apache.org/jira/browse/BEAM-1566?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15886435#comment-15886435
     ] 
    
    ASF GitHub Bot commented on BEAM-1566:
    --
    
    Github user asfgit closed the pull request at:
    
    https://github.com/apache/beam/pull/2116
    
    
    > Flink doesn't support DEFLATE compression
    > -
    >
    > Key: BEAM-1566
    > URL: https://issues.apache.org/jira/browse/BEAM-1566
    > Project: Beam
    >  Issue Type: Bug
    >  Components: runner-flink
    >Reporter: Daniel Halperin
    >Assignee: Daniel Halperin
    >
    > Adding support to FileBasedSink/Source for DEFLATE compression broke Flink 
    > because Flink overrides Apache Commons Compression down from 1.9 to 1.4.1.
    > I think the upgrade is safe, because Commons is general well behaved.
    
    
    
    --
    This message was sent by Atlassian JIRA
    (v6.3.15#6346)
    
    </pre></span>
    </blockquote><br>
    
    <h3><span class=subject><a href="/commits@beam.apache.org/msg05174.html">[2/2] beam git commit: This closes #2116</a></span></h3>
    <div class="darkgray font13">
    <span class="sender pipe">
    <span class=date><a href="/search?l=commits%40beam.apache.org&q=date:20170227&o=newest&f=1">2017-02-27</a></span></span>
    <span class="sender pipe">
    <span class=thead><a href="/search?l=commits%40beam.apache.org&q=subject:%22%5C%5B2%5C%2F2%5C%5D+beam+git+commit%5C%3A+This+closes+%232116%22&o=newest&f=1">Thread</a></span></span>
    <span class=name><a href="/search?l=commits%40beam.apache.org&q=from:%22dhalperi%22&o=newest&f=1">dhalperi</a></span>
    </div>
    <blockquote><span class="msgFragment"><pre>
    This closes #2116
    
    
    Project: http://git-wip-us.apache.org/repos/asf/beam/repo
    Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/ae6860db
    Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/ae6860db
    Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/ae6860db
    
    Branch: refs/heads/master
    Commit: ae6860db50cfe665dd4a26f2438eb528a47ffa25
    Parents: 932bb82 223aaac
    Author: Dan Halperin 
    Authored: Mon Feb 27 12:03:51 2017 -0800
    Committer: Dan Halperin 
    Committed: Mon Feb 27 12:03:51 2017 -0800
    
    --
     runners/flink/runner/pom.xml | 9 +
     1 file changed, 9 insertions(+)
    --
    
    
    
    </pre></span>
    </blockquote><br>
        <h2>  1   <a href="/search?l=commits%40beam.apache.org&q=date%3A20170227&o=newest&start=100">2</a>   <a href="/search?l=commits%40beam.apache.org&q=date%3A20170227&o=newest&start=100" accesskey="n"> > </a> </h2>
      </div>
      <div class="aside" role="complementary">
        <div class="logo">
          <a href="/"><img src="/logo.png" width=247 height=88 alt="The Mail Archive"></a>
        </div>
        <h2> 1 - 100 of 195 matches</h2>
        <br>
        
    <ul><li><a href="/search?l=commits%40beam.apache.org&q=date%3A20170227&a=1&o=newest&f=1">Advanced search</a></li></ul>
    <form class="overflow" action="/search" method="get">
    <input type="hidden" name="l" value="commits@beam.apache.org">
    <label class="hidden" for="q">Search the list</label>
    <input class="submittext" type="text" id="q" name="q" placeholder="Search commits" value="date:20170227">
    <input class="submitbutton" id="submit" type="image" src="/submit.png" alt="Submit">
    </form>
    
        
        <div class="nav margintop" id="nav" role="navigation">
          <h2 class="hidden">
                                   Site Navigation
          </h2>
          <ul class="icons font16">
            <li class="icons-home"><a href="/">The Mail Archive home</a></li>
            <li class="icons-list">
              <a href="/commits@beam.apache.org" title="c" id="c">commits - all messages</a></li>
            <li class="icons-about">
              <a href="/commits@beam.apache.org/info.html">commits  - about the list</a></li>
            <li class="icons-expand"><a href="/search?l=commits%40beam.apache.org&q=date%3A20170227&o=newest" title="e" id="e">Expand</a></li>
          </ul>
        </div>
    
        <div class="listlogo margintopdouble">
          <h2 class="hidden">
      				Mail list logo
          </h2>
          
        </div>
      </div>
      <div class="footer" role="contentinfo">
        <h2 class="hidden">
    	        	      Footer information
        </h2>
        <ul>
          <li><a href="/">The Mail Archive home</a></li>
          <li><a href="/faq.html#newlist">Add your mailing list</a></li>
          <li><a href="/faq.html">FAQ</a></li>
          <li><a href="/faq.html#support">Support</a></li>
          <li><a href="/faq.html#privacy">Privacy</a></li>
        </ul>
      </div>
    <script language="javascript" type="text/javascript">
    document.onkeydown = NavigateThrough;
    function NavigateThrough (event)
    {
      if (!document.getElementById) return;
      if (window.event) event = window.event;
      if (event.target.tagName == 'INPUT') return;
      if (event.ctrlKey || event.metaKey) return;
      var link = null;
      switch (event.keyCode ? event.keyCode : event.which ? event.which : null) {
        case 69:
          link = document.getElementById ('e');
          break;
        }
      if (link && link.href) document.location = link.href;
    }
    </script>
    </body>
    </html>