leizhiyuan commented on issue #1412: URL: https://github.com/apache/rocketmq-clients/issues/1412#issuecomment-6307860192
Update: item 9 is verified as fixed by #1404, so no separate PR is needed. All defects found by the end-to-end test are now covered: - open PRs: #1407, #1409, #1411, #1414, #1416, #1420, #1421, #1423, #1425; - covered by #1381 / #1404: items 2, 4b, 8 and 9; - won't fix: item 12. **Item 9.** The consumer was configured locally with `fifo=false`, and the server pushes `fifo=true`. On `master` + #1404 + #1381 (+ #1407, #1414, #1423, #1425), all queue actors start FIFO workers. A lite message that fails twice is delivered as attempt 1, 2, 3, and the next message of the same lite topic waits until it succeeds. Before, it was `F#1, F#1, N#1, F#3`. Details are in the body. **CI.** The 3 red checks on the `master`-based Rust PRs all come from `master`: - stable clippy `for_kv_map` in `client.rs`; - the MSRV check, because `powerfmt v0.2.1` requires rustc 1.79; - `build-result`, which only aggregates the two above. They are not caused by those PRs. #1404 fixes both, so they should go green after it is merged and the PRs are rebased. Added to the body. **Merge order is unchanged:** #1407 → #1411 → #1414 → #1404 → #1381 → #1409 / #1423 / #1425. #1416, #1420 and #1421 can go in at any point. The PRs are ready for review. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
