From 8b235191a5357a97b70e1401db7e2c4be4cb6c04 Mon Sep 17 00:00:00 2001 From: Matthias Broecheler Date: Mon, 25 Nov 2024 19:28:58 -0800 Subject: [PATCH] updated comment on config option Signed-off-by: Matthias Broecheler --- .../main/java/com/datasqrl/config/ExplainConfigImpl.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/sqrl-tools/sqrl-config/src/main/java/com/datasqrl/config/ExplainConfigImpl.java b/sqrl-tools/sqrl-config/src/main/java/com/datasqrl/config/ExplainConfigImpl.java index 7b8cce756..5fe3752e3 100644 --- a/sqrl-tools/sqrl-config/src/main/java/com/datasqrl/config/ExplainConfigImpl.java +++ b/sqrl-tools/sqrl-config/src/main/java/com/datasqrl/config/ExplainConfigImpl.java @@ -39,11 +39,13 @@ public class ExplainConfigImpl implements PackageJson.ExplainConfig { @Default boolean physical = false; /** - * This setting is primarily used for testing to ensure that the output of explain is - * deterministic + * Sorts the nodes in the DAG by id (i.e. their internal name). + * This is set to true by default to ensure a deterministic order of the DAG plan + * that can be used for testing. + * Set to false if you need the original DAG order. */ @Default - boolean sorted = true; //TODO: set to false and overwrite in test case injector + boolean sorted = true; public ExplainConfigImpl(SqrlConfig sqrlConfig) { this(