Disable persisted queries in validate mode (#5013)

Summary:
Right now, when the CLI is run in valdiate mode with persisted queries enabled, the compiler fails with connection refused errors (when the persisting server is not available).

This issue is most evident in CI pipelines when generally the persisting server won't be available.

Example error message:
```
➜  accounts (main) relay-compiler --validate                                                                                                                                                                                                                      ✭
[INFO] Querying files to compile...
[INFO] [default] compiling...
[ERROR] Error: Network error: error trying to connect: tcp connect error: Connection refused (os error 111)
[ERROR] Compilation failed.
[ERROR] Unable to run relay compiler. Error details:
Failed to build:
 - Persisting operation(s) failed:
 - Network error: error trying to connect: tcp connect error: Connection refused (os error 111)
 ```

 But the expected behaviour is that the query persister must be disabled in validate mode.

This PR attempts to fix that.

Issue(s) closed: #5007

Pull Request resolved: #5013

Reviewed By: tyao1

Differential Revision: D76442518

Pulled By: evanyeung

fbshipit-source-id: 31006555d4f4b1a8fd9679dc67188884388dd0d2

Author: aryaniyaps

compiler/crates/relay-bin/src/main.rs

@@ -314,22 +314,27 @@ async fn handle_compiler_command(command: CompileCommand) -> Result<(), Error> {
 
     if command.validate {
         config.artifact_writer = Box::<ArtifactValidationWriter>::default();
-    }
-
-    config.create_operation_persister = Some(Box::new(|project_config| {
-        project_config.persist.as_ref().map(
-            |persist_config| -> Box<dyn OperationPersister + Send + Sync> {
-                match persist_config {
-                    PersistConfig::Remote(remote_config) => {
-                        Box::new(RemotePersister::new(remote_config.clone()))
-                    }
-                    PersistConfig::Local(local_config) => {
-                        Box::new(LocalPersister::new(local_config.clone()))
+        // Disable persister and persisted queries in validate mode
+        config.create_operation_persister = None;
+        for project_config in config.projects.values_mut() {
+            project_config.persist = None;
+        }
+    } else {
+        config.create_operation_persister = Some(Box::new(|project_config| {
+            project_config.persist.as_ref().map(
+                |persist_config| -> Box<dyn OperationPersister + Send + Sync> {
+                    match persist_config {
+                        PersistConfig::Remote(remote_config) => {
+                            Box::new(RemotePersister::new(remote_config.clone()))
+                        }
+                        PersistConfig::Local(local_config) => {
+                            Box::new(LocalPersister::new(local_config.clone()))
+                        }
                     }
-                }
-            },
-        )
-    }));
+                },
+            )
+        }));
+    }
 
     config.file_source_config = if should_use_watchman() {
         FileSourceKind::Watchman