[GitHub] [incubator-druid] clintropolis commented on issue #8339: Don't inadvertently exclude hadoop-client-core dependency

[GitBox <gi...@apache.org>]

clintropolis commented on issue #8339: Don't inadvertently exclude hadoop-client-core dependency
URL: https://github.com/apache/incubator-druid/pull/8339#issuecomment-524718117
 
 
   Hi @Dylan1312, apologies for the delay into looking at this, could you provide some additional details about which Hadoop distribution/version and index task configuration that lead to encountering this issue? 
   
   I don't seem to have a problem on indexing or historical side if I use the docker container we provide for the Hadoop quick-start tutorial, which admittedly is hardly a realistic test, but word on the street is that releases greater than 0.14.0 work for at least some versions of Hadoop besides the version we bundle by default (2.8.3).
   
   I pulled your branch, and I can't quite tell what this patch would change; in terms of raw jars, a build of this branch appears identical to current master. But, I am also not incredibly familiar with the Hadoop index task class loader stuff, so maybe I'm missing something important in that regards? I'll try to have a closer look and see if I can get a better idea of what specifically is going on in there.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org