[jira] [Commented] (ZOOKEEPER-4308) Flaky test: EagerACLFilterTest.testSetDataFail

["Kezhu Wang (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ZOOKEEPER-4308?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17516275#comment-17516275 ] 

Kezhu Wang commented on ZOOKEEPER-4308:
---------------------------------------

This test use `zkLeader.getLastLoggedZxid()` as `lastxid` while it connect to `connectedServer` which maybe different `zkLeader`. There is no happen-before between `zkLeader.getLastLoggedZxid()` and successful response from other server. The commit and response operation is performed asynchronous and branches for different servers. Use `connectedServer.getLastProcessedZxid()` should solve this.

> Flaky test: EagerACLFilterTest.testSetDataFail
> ----------------------------------------------
>
>                 Key: ZOOKEEPER-4308
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-4308
>             Project: ZooKeeper
>          Issue Type: Bug
>          Components: tests
>    Affects Versions: 3.8.0
>            Reporter: Ling Mao
>            Assignee: Kezhu Wang
>            Priority: Major
>
> URL:[https://github.com/apache/zookeeper/pull/1425/checks?check_run_id=2702959273]
> {code:java}
> [INFO] 
> 947[ERROR] Failures: 
> 948[ERROR]   EagerACLFilterTest.testSetDataFail:222->assertTransactionState:115 Server State: OBSERVING Check Enabled: true Transaction state on Leader after failed setData ==> expected: <4294967298> but was: <4294967299>
> 949[INFO] 
> 950[ERROR] Tests run: 2913, Failures: 1, Errors: 0, Skipped: 4
> 951[INFO] {code}



--
This message was sent by Atlassian Jira
(v8.20.1#820001)