[ 
https://issues.apache.org/jira/browse/BEAM-6283?focusedWorklogId=178078&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-178078
 ]

ASF GitHub Bot logged work on BEAM-6283:
----------------------------------------

                Author: ASF GitHub Bot
            Created on: 21/Dec/18 18:02
            Start Date: 21/Dec/18 18:02
    Worklog Time Spent: 10m 
      Work Description: mxm commented on issue #7333: [BEAM-6283] Convert 
PortableStateTimerTest and PortableExecutionTest to using PAssert
URL: https://github.com/apache/beam/pull/7333#issuecomment-449457338
 
 
   This has nothing to do with test parallelization. The test itself is writing 
to a static map concurrently from multiple tasks/threads. It just happens to 
occur more likely under load when there are multiple instances of this tests or 
other tests running. This was more visible for the PortableTimersExecutionTest 
where we previously fixed it via #7214.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Issue Time Tracking
-------------------

    Worklog Id:     (was: 178078)
    Time Spent: 40m  (was: 0.5h)

> Convert PortableStateTimerTest and PortableExecutionTest to using PAssert
> -------------------------------------------------------------------------
>
>                 Key: BEAM-6283
>                 URL: https://issues.apache.org/jira/browse/BEAM-6283
>             Project: Beam
>          Issue Type: Test
>          Components: runner-flink
>            Reporter: Maximilian Michels
>            Assignee: Maximilian Michels
>            Priority: Minor
>             Fix For: 2.10.0
>
>          Time Spent: 40m
>  Remaining Estimate: 0h
>
> The tests modify a map concurrently. They should synchronize on the map or be 
> converted to using PAssert, similar to PortableTimerTest.
> Just saw PortableStateTest fail here: 
> https://builds.apache.org/job/beam_PreCommit_Java_Phrase/508



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to