Convert groovy code (ad-hoc scripts mostly) to java and make groovy dependency optional (#1194)

Wojciech Kapcia (Tigase) opened 5 years ago

Relying on groovy can be problematic (even now they were quite delayed when it comes to supporting newer Java) and the benefits are not that significant.

What's more:

better maintenance (compile time checks)
would help us shave off roughly 20% od dependencies
would help reduce memory usage.
would make it easier to use graalvm and native-image

(initially targeted for 9.0.0, depends on eventual http-api adjustments)

For templating we could possibly use jte as it's efficient and small/without further dependencies

Affected items:

ad-hoc scripts
http-api

TODO:

remove java.desktop from jlink (https://tigase.dev/tigase/_server/tigase-xmpp-server-docker/~issues/23#IssueComment-128653)

Activities

Wojciech Kapcia (Tigase) commented 5 years ago

In addition it would help reduce memory usage.
Wojciech Kapcia (Tigase) batch edited 2 years ago
Name Previous Value Current Value
Iterations
empty
Candidate for next major release
Wojciech Kapcia (Tigase) added "Related" tigase/_server/tigase-http-api#122 1 month ago
Wojciech Kapcia (Tigase) added "Related" #1564 1 month ago

Name	Previous Value	Current Value
Iterations	empty	Candidate for next major release

Wojciech Kapcia (Tigase) changed title 1 month ago

Previous Value	Current Value
Convert groovy code to java and make groovy dependency optional	Convert groovy code (ad-hoc scripts mostly) to java and make groovy dependency optional

Wojciech Kapcia (Tigase) referenced from other issue 1 month ago

Rewrite admin ad-hocs from Groovy to Java, improve ACL (#1570)

Open
Wojciech Kapcia (Tigase) added "Related" #1570 1 month ago
Wojciech Kapcia (Tigase) referenced from other issue 1 month ago

SessMan and VHostMan not visible for non-admin users (#1564)

In QA
Wojciech Kapcia (Tigase) commented 4 weeks ago

One more reason to remove groovy: it adds requirement to include java.desktop module, which adds 15M to our jlink/docker images (https://tigase.dev/tigase/_server/tigase-xmpp-server-docker/~issues/23#IssueComment-128653)
Login to comment

Type	Task
Priority	Normal
Assignee	Wojciech Kapcia (Tigase)
Version	Candidate for next major release

Iterations

Candidate for next major release Open

Issue Votes (0)

Watchers (2)

Reference

tigase/_server/server-core#1194