You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by GitBox <gi...@apache.org> on 2020/02/27 02:51:10 UTC

[GitHub] [beam] rohdesamuel opened a new pull request #10982: Fix Multi TestStream assertion errors

rohdesamuel opened a new pull request #10982: Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982
 
 
   Change-Id: Ibbd99867c9bc96aaa7ca9d3f2874f3b1e75c2f98
   
   Fixes failures from: https://builds.apache.org/job/beam_PreCommit_Python_Cron/2443/#showFailuresLink
   
   resulting from PR/10934
   
   ------------------------
   
   Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:
   
    - [ ] [**Choose reviewer(s)**](https://beam.apache.org/contribute/#make-your-change) and mention them in a comment (`R: @username`).
    - [ ] Format the pull request title like `[BEAM-XXX] Fixes bug in ApproximateQuantiles`, where you replace `BEAM-XXX` with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
    - [ ] Update `CHANGES.md` with noteworthy changes.
    - [ ] If this contribution is large, please file an Apache [Individual Contributor License Agreement](https://www.apache.org/licenses/icla.pdf).
   
   See the [Contributor Guide](https://beam.apache.org/contribute) for more tips on [how to make review process smoother](https://beam.apache.org/contribute/#make-reviewers-job-easier).
   
   Post-Commit Tests Status (on master branch)
   ------------------------------------------------------------------------------------------------
   
   Lang | SDK | Apex | Dataflow | Flink | Gearpump | Samza | Spark
   --- | --- | --- | --- | --- | --- | --- | ---
   Go | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Go/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Go/lastCompletedBuild/) | --- | --- | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Go_VR_Flink/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Go_VR_Flink/lastCompletedBuild/) | --- | --- | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Go_VR_Spark/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Go_VR_Spark/lastCompletedBuild/)
   Java | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Apex/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Apex/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Java11/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Java11/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Flink_Batch/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Flink_Batch/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Flink_Streaming/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Flink_Streaming/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Gearpump/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Gearpump/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Spark_Batch/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Spark_Batch/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_SparkStructuredStreaming/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_SparkStructuredStreaming/lastCompletedBuild/)
   Python | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Python2/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Python2/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Python35/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Python35/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Python36/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Python36/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Python37/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Python37/lastCompletedBuild/) | --- | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PreCommit_Python2_PVR_Flink_Cron/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PreCommit_Python2_PVR_Flink_Cron/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PostCommit_Python35_VR_Flink/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Python35_VR_Flink/lastCompletedBuild/) | --- | --- | [![Build Status](https://builds.apache.org/job/beam_PostCommit_Python_VR_Spark/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_Python_VR_Spark/lastCompletedBuild/)
   XLang | --- | --- | --- | [![Build Status](https://builds.apache.org/job/beam_PostCommit_XVR_Flink/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_XVR_Flink/lastCompletedBuild/) | --- | --- | [![Build Status](https://builds.apache.org/job/beam_PostCommit_XVR_Spark/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PostCommit_XVR_Spark/lastCompletedBuild/)
   
   Pre-Commit Tests Status (on master branch)
   ------------------------------------------------------------------------------------------------
   
   --- |Java | Python | Go | Website
   --- | --- | --- | --- | ---
   Non-portable | [![Build Status](https://builds.apache.org/job/beam_PreCommit_Java_Cron/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PreCommit_Java_Cron/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PreCommit_Python_Cron/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PreCommit_Python_Cron/lastCompletedBuild/)<br>[![Build Status](https://builds.apache.org/job/beam_PreCommit_PythonLint_Cron/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PreCommit_PythonLint_Cron/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PreCommit_Go_Cron/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PreCommit_Go_Cron/lastCompletedBuild/) | [![Build Status](https://builds.apache.org/job/beam_PreCommit_Website_Cron/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PreCommit_Website_Cron/lastCompletedBuild/) 
   Portable | --- | [![Build Status](https://builds.apache.org/job/beam_PreCommit_Portable_Python_Cron/lastCompletedBuild/badge/icon)](https://builds.apache.org/job/beam_PreCommit_Portable_Python_Cron/lastCompletedBuild/) | --- | ---
   
   See [.test-infra/jenkins/README](https://github.com/apache/beam/blob/master/.test-infra/jenkins/README.md) for trigger phrase, status and link of all Jenkins jobs.
   

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


With regards,
Apache Git Services

[GitHub] [beam] aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592282786
 
 
   Run Python PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] mwalenia merged pull request #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
mwalenia merged pull request #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982
 
 
   

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


With regards,
Apache Git Services

[GitHub] [beam] mwalenia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
mwalenia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592464294
 
 
   LGTM. Thanks for the fix!

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


With regards,
Apache Git Services

[GitHub] [beam] rohdesamuel commented on issue #10982: Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
rohdesamuel commented on issue #10982: Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-591752255
 
 
   R: @aaltay 

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592187719
 
 
   Run PythonLint PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592149742
 
 
   Run PythonLint PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592414688
 
 
   Run Python PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592220334
 
 
   retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] rohdesamuel commented on issue #10982: Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
rohdesamuel commented on issue #10982: Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-591752376
 
 
   CC: @lukecwik 

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


With regards,
Apache Git Services

[GitHub] [beam] pabloem commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
pabloem commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592136551
 
 
   Run PythonLint PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592203342
 
 
   retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592181684
 
 
   Run Python PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592297867
 
 
   @rohdesamuel precommit is still failing.

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592105966
 
 
   Run PythonLint PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] rohdesamuel commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
rohdesamuel commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592091176
 
 
   > So, wherever `TestStream` is used, `passthrough_pcollection_output_ids` needs to be specified. Is this correct?
   > 
   > Should we check in TestStream or add a note about it?
   
   Yes, specifically for the multiple output TestStream. I added a comment about it and which flag to use.

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


With regards,
Apache Git Services

[GitHub] [beam] aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592149918
 
 
   retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592427742
 
 
   `pycommon:docs` task is failing due to: `docstring of apache_beam.testing.test_stream.TestStream:14: WARNING: Unexpected indentation. `
   
   

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


With regards,
Apache Git Services

[GitHub] [beam] aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592154205
 
 
   retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592219819
 
 
   Run Python2_PVR_Flink PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592439340
 
 
   Retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] pabloem commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
pabloem commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592092078
 
 
   Restest this please

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


With regards,
Apache Git Services

[GitHub] [beam] yifanzou commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
yifanzou commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592138186
 
 
   > > The linter failed due to "ERROR: Could not install packages due to an EnvironmentError: [Errno 28] No space left on device"
   > 
   > @alanmyrvold @udim @yifanzou -- Is this a symptom of the problem of growing temp directory?
   
   The Jenkins workspace and the docker images are usually the two monsters eat the disk.

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia edited a comment on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia edited a comment on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592113465
 
 
   Run PythonLint PreCommit
   
   
   trying to hit another Jenkins.

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592244984
 
 
   retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592650626
 
 
   Thank you all!

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


With regards,
Apache Git Services

[GitHub] [beam] aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
aaltay commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592107642
 
 
   > The linter failed due to "ERROR: Could not install packages due to an EnvironmentError: [Errno 28] No space left on device"
   
   @alanmyrvold @udim @yifanzou  -- Is this a symptom of the problem of growing temp directory?

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592222533
 
 
   retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592187646
 
 
   retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592461386
 
 
   Tests passed. @mwalenia could you take a look?

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


With regards,
Apache Git Services

[GitHub] [beam] rohdesamuel commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
rohdesamuel commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592103975
 
 
   The linter failed due to "ERROR: Could not install packages due to an EnvironmentError: [Errno 28] No space left on device"

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592262649
 
 
   Run Python PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592226666
 
 
   retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] rohdesamuel commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
rohdesamuel commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592136428
 
 
   Run PythonLint PreCommit

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


With regards,
Apache Git Services

[GitHub] [beam] kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
kamilwu commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592438613
 
 
   Retest this please

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


With regards,
Apache Git Services

[GitHub] [beam] amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors

Posted by GitBox <gi...@apache.org>.
amaliujia commented on issue #10982: [BEAM-9392] Fix Multi TestStream assertion errors
URL: https://github.com/apache/beam/pull/10982#issuecomment-592113465
 
 
   Run PythonLint PreCommit

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


With regards,
Apache Git Services