[GitHub] [beam] lostluck commented on a change in pull request #15659: Disable samza counters

[GitBox <gi...@apache.org>]

lostluck commented on a change in pull request #15659:
URL: https://github.com/apache/beam/pull/15659#discussion_r722405441



##########
File path: sdks/go/test/integration/integration.go
##########
@@ -88,13 +88,17 @@ var flinkFilters = []string{
 }
 
 var samzaFilters = []string{
+	// XLang tests no supported by samza runner.
+	"TestXLang.*",
 	// TODO(BEAM-12608): Samza tests invalid encoding.
 	"TestReshuffle",
 	"TestReshuffleKV",
 	// The Samza runner does not support the TestStream primitive
 	"TestTestStream.*",
 	// The trigger tests uses TestStream
 	"TestTrigger.*",
+	// Samza doesn't yet support post job metrics, used by WordCount
+	"TestWordCount.*",

Review comment:
       I mean, we always can JIRA's are cheap. It's generally faster in cases like this to look for an existing one, and if not, create one, and then provide it than asking someone else to do it. In this case I've created it, because it's not worth the discussion time. BEAM-13006




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