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

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/06/27 17:02:24 UTC

[GitHub] [beam] damccorm commented on a diff in pull request #22065: Add randomness to integration test job names to avoid collisions

damccorm commented on code in PR #22065:
URL: https://github.com/apache/beam/pull/22065#discussion_r907604615


##########
sdks/go/test/integration/integration.go:
##########
@@ -214,7 +216,9 @@ func CheckFilters(t *testing.T) {
 		}
 	}
 	// TODO(lostluck): Improve default job names.

Review Comment:
   I didn't originally get rid of this because I wasn't sure if this fully completed it, but at the same time it is probably not helpful if its unclear so I'm good to remove it



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org