[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-28 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/1013 @arina-ielchiieva I guess just logging an exception does not help unless it targets Java developers. Consider logging a message that has information regarding which authentication provider can't be

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-28 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/1013 @vrozov as you can see from the changes, Volodymyr just has added additional information to the error message. I've seen from the previous discussions that you suggest just to log this

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-28 Thread vrozov
Github user vrozov commented on the issue: https://github.com/apache/drill/pull/1013 @vladimirtkach Who is the target of the message? If it is Java developers, the message makes sense. If it is for drill users or devops, the message is useless. ---

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-28 Thread vladimirtkach
Github user vladimirtkach commented on the issue: https://github.com/apache/drill/pull/1013 @sohami I'm getting this stack ``` java.lang.ExceptionInInitializerError at

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-27 Thread sohami
Github user sohami commented on the issue: https://github.com/apache/drill/pull/1013 I tried with the repro step and based on that doesn't see `DrillRuntimeException` message in stack trace but since the PR is adding the factory name in same exception message, we will not see that in

[GitHub] drill issue #1013: DRILL-5910: Add factory name to ClassNotFoundException me...

2017-10-27 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/1013 +1, LGTM. ---