Skip to content

Exposed the existing "disableAutoSync" property of ClusterNode to external configuration.#243

Open
tduffey wants to merge 2 commits into
apache:trunkfrom
tduffey:JCR-5149
Open

Exposed the existing "disableAutoSync" property of ClusterNode to external configuration.#243
tduffey wants to merge 2 commits into
apache:trunkfrom
tduffey:JCR-5149

Conversation

@tduffey

@tduffey tduffey commented May 15, 2025

Copy link
Copy Markdown

With this change you can now define your cluster config with an additional param to disable auto sync:

  <Cluster syncDelay="2000" disableAutoSync="true">
    <Journal class="org.apache.jackrabbit.core.journal.DatabaseJournal">
      ...
    </Journal>
  </Cluster>

With this we now have the ability to start a cluster node, allow it to synchronize to the latest journal revision at startup and then stop without worrying about any additional changes being sync'd via the normal journal sync process. The use case is to define a node like this to periodically launch, get the latest updates, stop and then save a backup.

I would be happy to provide a different unit test if you can suggest what sort of test you'd like to see since this is a very minimal change.

@tduffey

tduffey commented May 27, 2025

Copy link
Copy Markdown
Author

@thomasmueller any thoughts on this one? Accomplishes most of what I was after on the other one but with fewer actual code changes.

@thomasmueller

Copy link
Copy Markdown
Member

This looks good to me.

@tduffey

tduffey commented Jun 3, 2025

Copy link
Copy Markdown
Author

Thank you @thomasmueller. If this can be merged into a future release I would really appreciate it. 🙏

@tduffey

tduffey commented Jul 10, 2025

Copy link
Copy Markdown
Author

👋 @reschke any chance this can be incorporated into a future release?

@reschke

reschke commented Apr 1, 2026

Copy link
Copy Markdown
Contributor

@tduffey - can you open a JIRA issue for this? (see https://selfserve.apache.org/jira-account.html) - and sorry for letting you wait for so long.

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.

3 participants