You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by da...@apache.org on 2022/09/06 17:20:30 UTC

[beam] branch master updated: Disable singleIterate (#23042)

This is an automated email from the ASF dual-hosted git repository.

damccorm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new b2a6f46fb21 Disable singleIterate (#23042)
b2a6f46fb21 is described below

commit b2a6f46fb21709cc1927ce1950a38a922dce0a35
Author: Danny McCormick <da...@google.com>
AuthorDate: Tue Sep 6 13:20:21 2022 -0400

    Disable singleIterate (#23042)
    
    * Disable singleIterate
    
    * Comment with TODO
---
 sdks/go/pkg/beam/core/runtime/exec/datasource.go | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/sdks/go/pkg/beam/core/runtime/exec/datasource.go b/sdks/go/pkg/beam/core/runtime/exec/datasource.go
index dc44470228c..3e5084e857d 100644
--- a/sdks/go/pkg/beam/core/runtime/exec/datasource.go
+++ b/sdks/go/pkg/beam/core/runtime/exec/datasource.go
@@ -78,7 +78,8 @@ func (n *DataSource) ID() UnitID {
 
 // Up initializes this datasource.
 func (n *DataSource) Up(ctx context.Context) error {
-	safeToSingleIterate := true
+	// TODO(https://github.com/apache/beam/issues/23043) - Reenable single iteration or more fully rip this out.
+	safeToSingleIterate := false
 	switch n.Out.(type) {
 	case *Expand, *Multiplex:
 		// CoGBK Expands aren't safe, as they may re-iterate the GBK stream.