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 prestodb#23596
  • Loading branch information
BryanCutler committed Dec 3, 2024
1 parent 4479bcc commit cc94eae
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 cc94eae

Please sign in to comment.