Skip to content

Conversation

@Arnei
Copy link
Member

@Arnei Arnei commented Jan 8, 2026

Fixes #6705.

These beans seem to go unused and there are performance concerns surrounding the jmx statistics. Therefore this patch removes all jmx beans, to make our code a little bit simpler.

Your pull request should…

Arnei added 2 commits January 8, 2026 13:44
Not only are these going unused, but are also said to
cause performance issues. Therefore they can go.
They do not seem to serve any purposes, so we might
as well simplify our code.
@gregorydlogan
Copy link
Member

@lkiesow I have the vaguest recollection that you argued against removing JMX beans previously, am I right?
@rute-santos Is this relevant to your site?

@mtneug
Copy link
Member

mtneug commented Jan 13, 2026

JMX for OC job statistics can still be a performance issue during startup [1]. So +1 to remove JMX at least for that part.

[1] While it is possible to set the job statistic aggregation to only consider 1 day, the default is still 14. IIRC JMX is initialized before the config is read, i.e. even if 1 day is configured, during startup the default of 14 still applies. This will delay the whole startup of Opencast potentially waiting for a very long query. I don't think it is worth doing that given that JMX isn't used by (most/all?) adopters.

@JamesUoM
Copy link
Contributor

+1 for me, JMX statistics has never been anything but a performance issue.

Copy link
Member

@gregorydlogan gregorydlogan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Considering there's always (that person)[https://xkcd.com/1172/] out there somewhere, this should have a release notes entry too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove JMX statistics

4 participants