[
https://issues.apache.org/jira/browse/BEAM-8977?focusedWorklogId=361088&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-361088
]
ASF GitHub Bot logged work on BEAM-8977:
----------------------------------------
Author: ASF GitHub Bot
Created on: 17/Dec/19 19:52
Start Date: 17/Dec/19 19:52
Worklog Time Spent: 10m
Work Description: KevinGG commented on pull request #10404: [BEAM-8977]
Resolve test flakiness
URL: https://github.com/apache/beam/pull/10404
1. Removed test logic depending on execution of asynchronous tasks since
there is no control of them in a testing environment.
2. Replaced the dynamic plotting tests with tests directly/indirectly
invoking underlying logic of the asynchronous task.
------------------------
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.
- [ ] 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 | [](https://builds.apache.org/job/beam_PostCommit_Go/lastCompletedBuild/)
| --- | --- | [](https://builds.apache.org/job/beam_PostCommit_Go_VR_Flink/lastCompletedBuild/)
| --- | --- | [](https://builds.apache.org/job/beam_PostCommit_Go_VR_Spark/lastCompletedBuild/)
Java | [](https://builds.apache.org/job/beam_PostCommit_Java/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Apex/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Flink/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Flink_Batch/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Flink_Streaming/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Gearpump/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Samza/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Spark/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Java_PVR_Spark_Batch/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Java_ValidatesRunner_SparkStructuredStreaming/lastCompletedBuild/)
Python | [](https://builds.apache.org/job/beam_PostCommit_Python2/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Python35/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Python36/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Python37/lastCompletedBuild/)
| --- | [](https://builds.apache.org/job/beam_PostCommit_Py_VR_Dataflow/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Py_ValCont/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PreCommit_Python2_PVR_Flink_Cron/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PostCommit_Python35_VR_Flink/lastCompletedBuild/)
| --- | --- | [](https://builds.apache.org/job/beam_PostCommit_Python_VR_Spark/lastCompletedBuild/)
XLang | --- | --- | --- | [](https://builds.apache.org/job/beam_PostCommit_XVR_Flink/lastCompletedBuild/)
| --- | --- | ---
Pre-Commit Tests Status (on master branch)
------------------------------------------------------------------------------------------------
--- |Java | Python | Go | Website
--- | --- | --- | --- | ---
Non-portable | [](https://builds.apache.org/job/beam_PreCommit_Java_Cron/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PreCommit_Python_Cron/lastCompletedBuild/)<br>[](https://builds.apache.org/job/beam_PreCommit_PythonLint_Cron/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PreCommit_Go_Cron/lastCompletedBuild/)
| [](https://builds.apache.org/job/beam_PreCommit_Website_Cron/lastCompletedBuild/)
Portable | --- | [](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:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 361088)
Remaining Estimate: 0h
Time Spent: 10m
> apache_beam.runners.interactive.display.pcoll_visualization_test.PCollectionVisualizationTest.test_dynamic_plotting_update_same_display
> is flaky
> ------------------------------------------------------------------------------------------------------------------------------------------------
>
> Key: BEAM-8977
> URL: https://issues.apache.org/jira/browse/BEAM-8977
> Project: Beam
> Issue Type: Bug
> Components: test-failures
> Reporter: Valentyn Tymofieiev
> Assignee: Ning Kang
> Priority: Major
> Time Spent: 10m
> Remaining Estimate: 0h
>
> Sample failure:
>
> [https://builds.apache.org/job/beam_PreCommit_Python_Phrase/1273/testReport/apache_beam.runners.interactive.display.pcoll_visualization_test/PCollectionVisualizationTest/test_dynamic_plotting_update_same_display/]
> Error Message
> IndexError: list index out of range
> Stacktrace
> self =
> <apache_beam.runners.interactive.display.pcoll_visualization_test.PCollectionVisualizationTest
> testMethod=test_dynamic_plotting_update_same_display>
> mocked_display_facets = <MagicMock name='display_facets'
> id='139889868386376'>
> @patch('apache_beam.runners.interactive.display.pcoll_visualization'
> '.PCollectionVisualization.display_facets')
> def test_dynamic_plotting_update_same_display(self,
> mocked_display_facets):
> fake_pipeline_result = runner.PipelineResult(runner.PipelineState.RUNNING)
> ie.current_env().set_pipeline_result(self._p, fake_pipeline_result)
> # Starts async dynamic plotting that never ends in this test.
> h = pv.visualize(self._pcoll, dynamic_plotting_interval=0.001)
> # Blocking so the above async task can execute some iterations.
> time.sleep(1)
> # The first iteration doesn't provide updating_pv to display_facets.
> _, first_kwargs = mocked_display_facets.call_args_list[0]
> self.assertEqual(first_kwargs, {})
> # The following iterations use the same updating_pv to display_facets and so
> # on.
> > _, second_kwargs = mocked_display_facets.call_args_list[1]
> E IndexError: list index out of range
> apache_beam/runners/interactive/display/pcoll_visualization_test.py:105:
> IndexError
> Standard Output
> <IPython.core.display.HTML object>
> Standard Error
> WARNING:apache_beam.runners.interactive.interactive_environment:You cannot
> use Interactive Beam features when you are not in an interactive environment
> such as a Jupyter notebook or ipython terminal.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
