Skip to content

Commit

Permalink
Revert removal of SessionPropertyManager loading
Browse files Browse the repository at this point in the history
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
  • Loading branch information
BryanCutler committed Dec 3, 2024
1 parent ae16c69 commit 0213802
Showing 1 changed file with 2 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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);
Expand Down

0 comments on commit 0213802

Please sign in to comment.