This is an automated email from the ASF dual-hosted git repository.

mck pushed a commit to branch cassandra-4.0
in repository https://gitbox.apache.org/repos/asf/cassandra.git


The following commit(s) were added to refs/heads/cassandra-4.0 by this push:
     new 4555b94345 ninja-fix FullQueryLoggerTest, requires 
allow_nodetool_archive_command false to begin with CASSANDRA-18550
4555b94345 is described below

commit 4555b94345aa7183902bf1b5a54ad40fce13d432
Author: Mick Semb Wever <[email protected]>
AuthorDate: Fri May 26 13:40:51 2023 +0200

    ninja-fix FullQueryLoggerTest, requires allow_nodetool_archive_command 
false to begin with CASSANDRA-18550
---
 test/unit/org/apache/cassandra/fql/FullQueryLoggerTest.java | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/test/unit/org/apache/cassandra/fql/FullQueryLoggerTest.java 
b/test/unit/org/apache/cassandra/fql/FullQueryLoggerTest.java
index adb3ab67fd..712f0496e0 100644
--- a/test/unit/org/apache/cassandra/fql/FullQueryLoggerTest.java
+++ b/test/unit/org/apache/cassandra/fql/FullQueryLoggerTest.java
@@ -680,6 +680,7 @@ public class FullQueryLoggerTest extends CQLTester
 
         try
         {
+            
DatabaseDescriptor.getFullQueryLogOptions().allow_nodetool_archive_command = 
false;
             StorageService.instance.enableFullQueryLogger(options.log_dir, 
options.roll_cycle, false, 1000, 1000, "/xyz/not/null", 0);
             fail("not allowed");
         }
@@ -688,6 +689,7 @@ public class FullQueryLoggerTest extends CQLTester
             assertTrue(e.getMessage().contains("Can't enable full query log 
archiving via nodetool"));
         }
 
+        options.allow_nodetool_archive_command = true;
         options.archive_command = "/xyz/not/null";
         options.log_dir = "/tmp/abc";
         DatabaseDescriptor.setFullQueryLogOptions(options);


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to