From fdf115940bf00ed33c19741538bac76c2474b0da Mon Sep 17 00:00:00 2001
From: Zoltan Haindrich <kirk@rxd.hu>
Date: Tue, 18 Jun 2024 16:43:10 +0000
Subject: [PATCH] disable record by default

---
 quidem-it/README.md | 6 ++++--
 quidem-it/pom.xml   | 3 ++-
 2 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/quidem-it/README.md b/quidem-it/README.md
index c800815599d..a53d4164783 100644
--- a/quidem-it/README.md
+++ b/quidem-it/README.md
@@ -19,7 +19,7 @@ Can be used to write tests against existing test backends (ComponentSupplier) -
   ```
 * launch the broker instance with:
   ```
-  mvn exec:exec -pl quidem-it
+  mvn exec:exec -pl quidem-it -Dquidem.record.autostart=true
   ```
   * the broker will be running at http://localhost:12345
   * the used test configuration backend can configured by supplying `quidem.uri`
@@ -27,6 +27,8 @@ Can be used to write tests against existing test backends (ComponentSupplier) -
     mvn exec:exec -pl quidem-it -Dquidem.uri=druidtest:///?componentSupplier=ThetaSketchComponentSupplier
     ``` 
   * new record files can be started by calling http://localhost:12345/quidem/start
+    * if `quidem.record.autostart` is omitted recording will not start
 * after finished with the usage of the broker ; a new `record-123.iq` file will be in the project's worktree - it will contain all the executed statements
-  * running `dev/quidem -Dquidem.overwrite` adds the resultsets to the `iq` file
+  * running `dev/quidem -Dquidem.overwrite` updates the resultsets of all  `iq` files around there
+  * rename the testfile to have a more descriptive name
 
diff --git a/quidem-it/pom.xml b/quidem-it/pom.xml
index 5d703f8f6d7..cdbaa7b2045 100644
--- a/quidem-it/pom.xml
+++ b/quidem-it/pom.xml
@@ -42,6 +42,7 @@
 
     <properties>
         <quidem.uri>druidtest:///</quidem.uri>
+        <quidem.record.autostart>false</quidem.record.autostart>
     </properties>
     <dependencies>
         <dependency>
@@ -512,7 +513,7 @@
                         <argument>-classpath</argument>
                         <classpath />
                         <argument>-Dquidem.uri=${quidem.uri}</argument>
-                        <argument>-Dquidem.record.autostart=true</argument>
+                        <argument>-Dquidem.record.autostart=${quidem.record.autostart}</argument>
                         <argument>org.apache.druid.quidem.Launcher</argument>
                     </arguments>
                 </configuration>