Skip to content

Commit d7c0930

Browse files
ericyuliurschlussel
authored andcommitted
Improve boostrap log to expose the wrong classLoader (#24533)
Summary: Improve bootstrap error message for incorrect classloader to include what classloader was found. Differential Revision: D69416153
1 parent db0fe87 commit d7c0930

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

presto-main/src/main/java/com/facebook/presto/sql/gen/Bootstrap.java

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -44,8 +44,12 @@ private Bootstrap()
4444
public static CallSite bootstrap(MethodHandles.Lookup callerLookup, String name, MethodType type, long bindingId)
4545
{
4646
ClassLoader classLoader = callerLookup.lookupClass().getClassLoader();
47-
checkArgument(classLoader instanceof DynamicClassLoader, "Expected %s's classloader to be of type %s", callerLookup.lookupClass().getName(), DynamicClassLoader.class.getName());
48-
47+
checkArgument(
48+
classLoader instanceof DynamicClassLoader,
49+
"Expected %s's classloader to be of type %s, but is %s",
50+
callerLookup.lookupClass().getName(),
51+
DynamicClassLoader.class.getName(),
52+
classLoader.getClass().getName());
4953
DynamicClassLoader dynamicClassLoader = (DynamicClassLoader) classLoader;
5054
MethodHandle target = dynamicClassLoader.getCallSiteBindings().get(bindingId);
5155
checkArgument(target != null, "Binding %s for function %s%s not found", bindingId, name, type.parameterList());

0 commit comments

Comments
 (0)