----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39430/#review103096 -----------------------------------------------------------
lens-ship-jars/pom.xml (line 70) <https://reviews.apache.org/r/39430/#comment160983> Can we include open csv aswell, which is right now coming as transitive dependency from lens-query-lib? - Amareshwari Sriramadasu On Oct. 19, 2015, 4:46 a.m., Srikanth Sundarrajan wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39430/ > ----------------------------------------------------------- > > (Updated Oct. 19, 2015, 4:46 a.m.) > > > Review request for lens. > > > Bugs: LENS-831 > https://issues.apache.org/jira/browse/LENS-831 > > > Repository: lens > > > Description > ------- > > Currently a number of jars have to be shipped along with each query apart > from user extension jars for udf, serde etc. This has two side effects: > > 1. Number of ClassLoader on the hive server increases (potentially causing > PermGen OOM issues) > 2. Number of jar localization events in the node manager when executing a > task. > > The proposal is to bundle these and create a single shaded jar and make it > available in the binary distribution. > > > Diffs > ----- > > lens-dist/pom.xml bc94209 > lens-dist/src/main/assembly/bin-dist.xml 8707af9 > lens-ship-jars/LICENSE PRE-CREATION > lens-ship-jars/pom.xml PRE-CREATION > lens-ship-jars/src/site/apt/index.apt PRE-CREATION > pom.xml 7140038 > src/site/apt/lenshome/install-and-run.apt 491bbee > > Diff: https://reviews.apache.org/r/39430/diff/ > > > Testing > ------- > > Verified that the shippable jar is created correctly and the same is being > added to the binary distribution package > > > Thanks, > > Srikanth Sundarrajan > >
