> On 一月 24, 2022, 9:52 a.m., Pradeep Agrawal wrote: > > Are you removing the web.xml permanently ?
There 2 web.xml in git, the effective content are same. kms/src/main/resources/WEB-INF/web.xml kms/src/main/webapp/WEB-INF/web.xml ``` % diff kms//src/main/resources/WEB-INF/web.xml kms/src/main/webapp/WEB-INF/web.xml 17a18 > 36,41d36 < < <!-- <servlet> < <servlet-name>RangerKMSStartUp</servlet-name> < <servlet-class>org.apache.ranger.kms.biz.RangerKMSStartUp</servlet-class> < <load-on-startup>2</load-on-startup> < </servlet> --> ``` I removed kms/src/main/resources/WEB-INF/web.xml which is not used by tomcat. - Kirby ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73816/#review223991 ----------------------------------------------------------- On 一月 24, 2022, 2:49 a.m., Kirby Zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73816/ > ----------------------------------------------------------- > > (Updated 一月 24, 2022, 2:49 a.m.) > > > Review request for ranger. > > > Bugs: RANGER-3595 > https://issues.apache.org/jira/browse/RANGER-3595 > > > Repository: ranger > > > Description > ------- > > https://issues.apache.org/jira/browse/RANGER-3595 > > > There are lots of .class files under ews/webapp/, and never used. > Becasue > 1. place web.xml at correct location > 2. setup.sh want to patch hadoop-common.jar at runtime, it requires some file > inside ranger-kms.jar. But the patching of hadoop-common.jar is unnecessary. > > My patch here makes maven no longer unpack ranger-kms.jar, and place web.xml > at correct location. > > > Diffs > ----- > > distro/src/main/assembly/kms.xml 32bbefa44e372f3abb41d60cd35aa0d706ca3100 > kms/scripts/setup.sh 2051df59a8bb0be11ba7a54e547f78cf5a0dca36 > kms/src/main/resources/WEB-INF/web.xml > 5e2d489fe632a16a5ed440bbbff41f558d2a2338 > > > Diff: https://reviews.apache.org/r/73816/diff/1/ > > > Testing > ------- > > mvn clean build package. > > fresh installion and upgrade. > > > Thanks, > > Kirby Zhou > >
