Copilot commented on code in PR #19259:
URL: https://github.com/apache/pinot/pull/19259#discussion_r3788631291
##########
pinot-server/src/main/java/org/apache/pinot/server/starter/ServerInstance.java:
##########
@@ -187,8 +199,6 @@ public ServerInstance(ServerConf serverConf, String
instanceId, HelixManager hel
}
Review Comment:
The ClassNotFoundException cause is dropped when rethrowing here, which
makes startup failures harder to debug (especially if the missing class name is
derived/configured). Preserve the original exception as the cause so stack
traces and suppressed context are retained.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
