[GitHub] [accumulo-proxy] volmasoft commented on pull request #23: accumulo-proxy #22 - Reduce Docker image size

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

volmasoft commented on pull request #23:
URL: https://github.com/apache/accumulo-proxy/pull/23#issuecomment-639542202


   > @volmasoft on the issue removing the hadoop docs folder was mentioned, but I did not see it.
   > 
   > Based on the [default Accumulo classpath](https://github.com/apache/accumulo/blob/rel/2.0.0/assemble/conf/accumulo-env.sh#L59), maybe everything except `${HADOOP_HOME}/share/hadoop/client/*` and the hadoop conf dir could be removed.
   
   I wasn't sure if it's allowable to remove it e.g. does the license restrict it, as I'm no expert in this arena I was hoping someone else would be able to provide guidance.
   
   Looking at the official license (https://apache.org/licenses/LICENSE-2.0) it appears that we should be fine (see 4. Redistribution)
   "4. Redistribution. You may reproduce and distribute copies of the Work or Derivative Works thereof in any medium, with or without modifications, and in Source or Object form, provided that You meet the following conditions:"
   
   So perhaps we can modify this but if so do we need to put any documentation/license information anywhere? 
   
   I didn't want to do something in good faith but potentially cause issues for us down the line so hence I haven't included that change currently.


----------------------------------------------------------------
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