You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@beam.apache.org by "Daniel Halperin (JIRA)" <ji...@apache.org> on 2017/04/07 17:59:41 UTC

[jira] [Created] (BEAM-1907) Delete PubsubBoundedReader

Daniel Halperin created BEAM-1907:
-------------------------------------

             Summary: Delete PubsubBoundedReader
                 Key: BEAM-1907
                 URL: https://issues.apache.org/jira/browse/BEAM-1907
             Project: Beam
          Issue Type: Improvement
          Components: sdk-java-gcp
            Reporter: Daniel Halperin
            Assignee: Daniel Halperin
             Fix For: First stable release


PubsubIO in bounded mode doesn't really make sense, outside of hacky testing modes -- it might lose data, other stuff that's sketchy. We had it before because the old {{DirectRunner}} did not support unbounded PCollections. Now that it does, we should probably get rid of this buggy code.

Specifically:

* Delete the specialized PubsubBoundedReader implementation
* Either: delete the maxNumRecords and maxReadTime methods, or rename them to something like maxNumRecordsForTesting with clear javadoc as to their drawbacks.



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