From cc94eaeb32b0fcf90e41c6248b41d83641bffb04 Mon Sep 17 00:00:00 2001
From: Bryan Cutler <cutlerb@gmail.com>
Date: Mon, 2 Dec 2024 21:49:51 -0800
Subject: [PATCH] Revert removal of SessionPropertyManager loading

The previous commit to add the native plan checker mistakenly removed
the loading of SessionPropertyManager. This adds it back the call to
loadSessionPropertyProviders into PrestoServer.

See #23596
---
 .../src/main/java/com/facebook/presto/server/PrestoServer.java  | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/presto-main/src/main/java/com/facebook/presto/server/PrestoServer.java b/presto-main/src/main/java/com/facebook/presto/server/PrestoServer.java
index 1c0ce0dff9bbc..d8a0fe050e7ad 100644
--- a/presto-main/src/main/java/com/facebook/presto/server/PrestoServer.java
+++ b/presto-main/src/main/java/com/facebook/presto/server/PrestoServer.java
@@ -42,6 +42,7 @@
 import com.facebook.presto.metadata.CatalogManager;
 import com.facebook.presto.metadata.DiscoveryNodeManager;
 import com.facebook.presto.metadata.InternalNodeManager;
+import com.facebook.presto.metadata.SessionPropertyManager;
 import com.facebook.presto.metadata.StaticCatalogStore;
 import com.facebook.presto.metadata.StaticFunctionNamespaceStore;
 import com.facebook.presto.nodeManager.PluginNodeManager;
@@ -182,6 +183,7 @@ public void run()
             injector.getInstance(TracerProviderManager.class).loadTracerProvider();
             injector.getInstance(NodeStatusNotificationManager.class).loadNodeStatusNotificationProvider();
             injector.getInstance(GracefulShutdownHandler.class).loadNodeStatusNotification();
+            injector.getInstance(SessionPropertyManager.class).loadSessionPropertyProviders();
             PlanCheckerProviderManager planCheckerProviderManager = injector.getInstance(PlanCheckerProviderManager.class);
             InternalNodeManager nodeManager = injector.getInstance(DiscoveryNodeManager.class);
             NodeInfo nodeInfo = injector.getInstance(NodeInfo.class);