[GitHub] drill issue #1002: DRILL-5888: Remove dependency of SSLConfig on hadoop.secu...

2017-10-20 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/1002 Yes, I will remove it. Thanks ! ---

[GitHub] drill issue #1002: DRILL-5888: Remove dependency of SSLConfig on hadoop.secu...

2017-10-20 Thread arina-ielchiieva
Github user arina-ielchiieva commented on the issue: https://github.com/apache/drill/pull/1002 One more thing, I have noticed that `TestSSLConfig#testInvalidHadoopKeystore()` contains the following line ``` String hadoopSSLFileProp = MessageFormat

[GitHub] drill issue #1002: DRILL-5888: Remove dependency of SSLConfig on hadoop.secu...

2017-10-19 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/1002 Thanks Arina. I removed the unused imports. ---

[GitHub] drill issue #1002: DRILL-5888: Remove dependency of SSLConfig on hadoop.secu...

2017-10-18 Thread parthchandra
Github user parthchandra commented on the issue: https://github.com/apache/drill/pull/1002 Yes. The property is consumed by SSLConfig which uses that to construct a key name that is looked up in the hadoop config. ---