This is an automated email from the ASF dual-hosted git repository. jlewandowski pushed a commit to branch ds-trunk-5.0--2024-07-24 in repository https://gitbox.apache.org/repos/asf/cassandra-dtest.git
commit 0e9ebad98dcf3a44670ead79c549b30bb7bf4c2d Author: Tomek Ĺasica <[email protected]> AuthorDate: Fri Jul 2 08:58:39 2021 +0200 STAR-517. Skip as it is not possible to test it reliably. (#34) The goal of the test is to "exercise the retry and back-off policies". And test is trying to achieve it by: - using arbitrary low write timeout - and loading / exporting / importing / exporting the data - assuming that retries and back-off will handle it. Unfortunately it is very sensitive... Because we need to find a perfect balance for the timeout and req/s, so that: - timeouts are indeed hit - and default COPY util not abort after 90s without succ write - and default COPY will not abort after 12 unsuccessull backoff attempts This change disables this test because: - there is no logic in the test that would check if indeed retries and backoff are exercised, so - it is perfectly possible that when it passes it just passes without any value - there are special tests for MAXATTEMPTS and for backoff policy Final note: mechanism of retries and backoff policy in COPY command could be improved by providing some backpressure mechanism to slowdown writes rather than doing retry or backoff exponentital (with random!). But this would be a new feature and COPY util is not a place we would like to invest. (cherry picked from commit 897aab709d133b6733bbb525ee65505b220e7971) (cherry picked from commit 602cf5329722d06cae68fe95325979d8fb8f47d0) (cherry picked from commit cb701f71244bda4d349a80ae7d98e153ceed0cda) (cherry picked from commit c04b7ded878a4e795fbe889a246888946fa7f13c) (cherry picked from commit 3b5125734fc45ad6f64ca2f9f2be9be7cd32ef9a) (cherry picked from commit 39fc3bd99bc655b3156c98a3bf62695903f3d628) --- cqlsh_tests/test_cqlsh_copy.py | 1 + 1 file changed, 1 insertion(+) diff --git a/cqlsh_tests/test_cqlsh_copy.py b/cqlsh_tests/test_cqlsh_copy.py index 458804c7..67ea0cdc 100644 --- a/cqlsh_tests/test_cqlsh_copy.py +++ b/cqlsh_tests/test_cqlsh_copy.py @@ -2514,6 +2514,7 @@ class TestCqlshCopy(Tester): copy_to_options={'NUMPROCESSES': 5, 'MAXATTEMPTS': 20}, copy_from_options={'NUMPROCESSES': 2}) + @pytest.mark.skip(reason="test is not reliable and sensitive to vm power") def test_bulk_round_trip_with_timeouts(self): """ Test bulk import with very short read and write timeout values, this should exercise the --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
