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 2020/12/08 16:56:11 UTC

[GitHub] [beam] kamilwu commented on a change in pull request #13494: Fix go sdk gbk load test

kamilwu commented on a change in pull request #13494:
URL: https://github.com/apache/beam/pull/13494#discussion_r538589685



##########
File path: .test-infra/jenkins/job_LoadTests_GBK_Flink_Go.groovy
##########
@@ -27,6 +27,10 @@ import static LoadTestsBuilder.DOCKER_CONTAINER_REGISTRY
 
 String now = new Date().format('MMddHHmmss', TimeZone.getTimeZone('UTC'))
 
+def TESTS_TO_SKIP = [

Review comment:
       You should explain why do you want to skip that test. I think one or two sentences describing the memory issue that you observed is fine. You could also add a TODO and attach it to this JIRA ticket: https://issues.apache.org/jira/browse/BEAM-9761 which tracks problems with GBK and CoGBK for Python (the issue can be possibly related)




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

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