review feedback: disable expensive batch write test for cassandra

scylladb · Oct 13, 2023 · a23e6eb · a23e6eb
1 parent ec5cf68
commit a23e6eb
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/.github/workflows/cassandra.yml b/.github/workflows/cassandra.yml
@@ -29,7 +29,7 @@ jobs:
       run: cargo build --verbose --tests
     - name: Run tests on cassandra
       run: |
-        CDC='disabled' SCYLLA_URI=172.42.0.2:9042 SCYLLA_URI2=172.42.0.3:9042 SCYLLA_URI3=172.42.0.4:9042 cargo test --verbose -- --skip test_views_in_schema_info
+        CDC='disabled' SCYLLA_URI=172.42.0.2:9042 SCYLLA_URI2=172.42.0.3:9042 SCYLLA_URI3=172.42.0.4:9042 cargo test --verbose -- --skip test_views_in_schema_info --skip test_large_batch_statements
     - name: Stop the cluster
       if: ${{ always() }}
       run: docker compose -f test/cluster/cassandra/docker-compose.yml stop

diff --git a/scylla/src/transport/large_batch_statements_test.rs b/scylla/src/transport/large_batch_statements_test.rs
@@ -17,6 +17,11 @@ async fn test_large_batch_statements() {
     let ks = unique_keyspace_name();
     session = create_test_session(session, &ks).await;
 
+    let max_queries = u16::MAX as usize;
+    let batch_insert_result = write_batch(&session, max_queries, &ks).await;
+
+    assert!(batch_insert_result.is_ok());
+
     let too_many_queries = u16::MAX as usize + 1;
     let batch_insert_result = write_batch(&session, too_many_queries, &ks).await;
     assert_matches!(