[GitHub] [incubator-druid] clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries

2019-08-16 Thread GitBox
clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries URL: https://github.com/apache/incubator-druid/pull/8310#issuecomment-522172439 final set of dependencies after this PR: ![Screen Shot 2019-08-16 at 3 39 25

[GitHub] [incubator-druid] clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries

2019-08-16 Thread GitBox
clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries URL: https://github.com/apache/incubator-druid/pull/8310#issuecomment-521907637 Tested in a kerberized docker environment, http requests were functioning but,

[GitHub] [incubator-druid] clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries

2019-08-15 Thread GitBox
clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries URL: https://github.com/apache/incubator-druid/pull/8310#issuecomment-521860451 Running some tests to make sure everything is still good after these changes

[GitHub] [incubator-druid] clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries

2019-08-15 Thread GitBox
clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries URL: https://github.com/apache/incubator-druid/pull/8310#issuecomment-521563375 Decided to just go ahead and try removing most of the ones intellij indicated we

[GitHub] [incubator-druid] clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries

2019-08-15 Thread GitBox
clintropolis commented on issue #8310: exclude kerberos extension dependencies that are already included in druid core libraries URL: https://github.com/apache/incubator-druid/pull/8310#issuecomment-521559781 Oops, looks like `json-smart` can be excluded, others look like issues not