You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@beam.apache.org by Jan Lukavský <je...@seznam.cz> on 2021/07/06 07:51:51 UTC
Re: JavaPrecommit fails
Looks to me to be failing consistently. At least appears so both on PRs
and tested locally on master branch.
On 7/5/21 1:34 PM, Alexey Romanenko wrote:
> Hello,
>
> JavaPrecommit fails because of “org.apache.beam.fn.harness.control.HarnessMonitoringInfosInstructionHandlerTest.testReturnsProcessWideMonitoringInfos” test.
> Is it a regression or just flaky?
>
> —
> Alexey
>
>
Re: JavaPrecommit fails
Posted by Alexey Romanenko <ar...@gmail.com>.
For the reference: https://issues.apache.org/jira/browse/BEAM-12583
> On 6 Jul 2021, at 18:31, Alexey Romanenko <ar...@gmail.com> wrote:
>
> Yes, it fails constantly for me too. I briefly did a "git bisect” and a60aebaac9c90d393ca1e2a7445d45222d908541 is the first bad commit. It’s quite strange that this test passed initially on Jenkins but it seems for me that PR #14805 [1] was tested without taking into account of changes from another one #14804 [2] in ShortIdMap.java where it fails now.
>
> @Alex Amato
> Could you take a look, please?
>
> —
> Alexey
>
> [1] https://github.com/apache/beam/pull/14805
> [2] https://github.com/apache/beam/pull/14804
>
>
>> On 6 Jul 2021, at 09:51, Jan Lukavský <je...@seznam.cz> wrote:
>>
>> Looks to me to be failing consistently. At least appears so both on PRs and tested locally on master branch.
>>
>> On 7/5/21 1:34 PM, Alexey Romanenko wrote:
>>> Hello,
>>>
>>> JavaPrecommit fails because of “org.apache.beam.fn.harness.control.HarnessMonitoringInfosInstructionHandlerTest.testReturnsProcessWideMonitoringInfos” test.
>>> Is it a regression or just flaky?
>>>
>>> —
>>> Alexey
>>>
>>>
>
Re: JavaPrecommit fails
Posted by Alexey Romanenko <ar...@gmail.com>.
Yes, it fails constantly for me too. I briefly did a "git bisect” and a60aebaac9c90d393ca1e2a7445d45222d908541 is the first bad commit. It’s quite strange that this test passed initially on Jenkins but it seems for me that PR #14805 [1] was tested without taking into account of changes from another one #14804 [2] in ShortIdMap.java where it fails now.
@Alex Amato
Could you take a look, please?
—
Alexey
[1] https://github.com/apache/beam/pull/14805
[2] https://github.com/apache/beam/pull/14804
> On 6 Jul 2021, at 09:51, Jan Lukavský <je...@seznam.cz> wrote:
>
> Looks to me to be failing consistently. At least appears so both on PRs and tested locally on master branch.
>
> On 7/5/21 1:34 PM, Alexey Romanenko wrote:
>> Hello,
>>
>> JavaPrecommit fails because of “org.apache.beam.fn.harness.control.HarnessMonitoringInfosInstructionHandlerTest.testReturnsProcessWideMonitoringInfos” test.
>> Is it a regression or just flaky?
>>
>> —
>> Alexey
>>
>>