Skip to content

Remove XStream from dependencies#5338

Open
knstvk wants to merge 1 commit into
masterfrom
5337-remove-xstream
Open

Remove XStream from dependencies#5338
knstvk wants to merge 1 commit into
masterfrom
5337-remove-xstream

Conversation

@knstvk
Copy link
Copy Markdown
Contributor

@knstvk knstvk commented Jun 1, 2026

XStream is now used only in legacy parts of Audit and Reports add-ons: XML-based storage of entity snapshots and reports. Not enough motivation to maintain a dependency, so let's drop these unused legacy storage format.

@knstvk knstvk linked an issue Jun 1, 2026 that may be closed by this pull request
@knstvk knstvk marked this pull request as ready for review June 1, 2026 14:17
@knstvk knstvk requested review from KremnevDmitry and glebfox June 1, 2026 14:18
Copy link
Copy Markdown
Contributor

@KremnevDmitry KremnevDmitry left a comment

Choose a reason for hiding this comment

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

  1. Credits should be removed too, I guess
Image
  1. Im not sure about naming of the following attributes of EntitySnapshot:
    @Column(name = "FETCH_PLAN_XML")
    private String fetchPlanXml;

    @Column(name = "SNAPSHOT_XML")
    private String snapshotXml;

We actually store JSON values in these properties.
Is renaming them part of the current issue, or should we leave everything as is?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove XStream from dependencies

3 participants