Move java dependency tomcat-embed to the crossTest configuration to remove outdated unnecessary compile time dependency.
This closes #2340
For the java libthrift, tomcat-embedded is only used in crossTests, I have moved it to crossTest configuration so the libthrift java package does not require this unnecessary dependency for compilation. Instead, the java-servlet dependency has been reintroduced in compile time. I've also taken this opportunity to update both dependenciesto a later version.
diff --git a/lib/java/gradle.properties b/lib/java/gradle.properties
index b87bdc7..7208807 100644
--- a/lib/java/gradle.properties
+++ b/lib/java/gradle.properties
@@ -28,9 +28,8 @@
httpclient.version=4.5.10
httpcore.version=4.4.12
slf4j.version=1.7.28
-#servlet.version=2.5
-#It contains servlet3
-tomcat.embed.version=8.5.46
+servlet.version=4.0.1
+tomcat.embed.version=9.0.43
junit.version=4.12
mockito.version=1.10.19
javax.annotation.version=1.3.2