[GitHub] [beam] lostluck commented on pull request #15743: [BEAM-11087] Add default WindowMappingFn from Main to Side Input windows, validation test, unit tests

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

lostluck commented on pull request #15743:
URL: https://github.com/apache/beam/pull/15743#issuecomment-948886943


   Ah very good find. Thank you.
   
   On Thu, Oct 21, 2021, 11:10 AM Lukasz Cwik ***@***.***> wrote:
   
   > It looks like the addition of TestValidateWindowedSideInputs bumped us
   > above the 1h timeout such that the Go PostCommit will timeout more often
   > (for example https://ci-beam.apache.org/job/beam_PostCommit_Go/9019/)
   >
   > It used to be that the run would take ~55mins.
   >
   > Checking the associated job (
   > https://console.cloud.google.com/dataflow/jobs/us-central1/2021-10-21_06_01_28-15917691559290441824?project=apache-beam-testing)
   > and it passed after about 8 mins so ~55 + ~8 > 1h.
   >
   > Filed https://issues.apache.org/jira/browse/BEAM-13096
   >
   > —
   > You are receiving this because you modified the open/close state.
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/beam/pull/15743#issuecomment-948878641>, or
   > unsubscribe
   > <https://github.com/notifications/unsubscribe-auth/ADKDOFLMB26SFBEWO46OB53UIBJP5ANCNFSM5GHK3EMQ>
   > .
   > Triage notifications on the go with GitHub Mobile for iOS
   > <https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
   > or Android
   > <https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
   >
   >
   


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