Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[MRESOLVER-278] On repository system shutdown #209

Merged
merged 1 commit into from
Oct 28, 2022

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Oct 27, 2022

Drops problematic onSessionClose, keeps only onSystemClose support. Issue will be adjusted accordingly.

Key changes:

  • undone all related to onSessionClose
  • kept onRepositorySystemClose + shutdown
  • all 3 users on onSessionClose (locking + 2 "recording" features) will act on shutdown
  • moved out of session.data use as it is completely unreliable, now singleton components hold caches (hence the shutdown persistence)
  • DefaultSyncContext: fixed as explained, does not use system properties anymore, but is made pretty much same as before [MRESOLVER-266] Simplify named lock adapter creation and align config source #196
  • renamed all new components to camelCase

https://issues.apache.org/jira/browse/MRESOLVER-278

@cstamas cstamas self-assigned this Oct 27, 2022
@cstamas cstamas force-pushed the MRESOLVER-278-onSessionClose-rework branch from 7c687a1 to 71ca943 Compare October 27, 2022 20:41
@cstamas cstamas marked this pull request as ready for review October 27, 2022 20:42
@cstamas cstamas requested review from michael-o and gnodet October 27, 2022 20:43
@cstamas cstamas force-pushed the MRESOLVER-278-onSessionClose-rework branch from 31d2a6f to cc4c213 Compare October 28, 2022 13:10
@cstamas cstamas changed the title [MRESOLVER-278] On session close reworked [MRESOLVER-278] On repository system shutdown Oct 28, 2022
@cstamas cstamas force-pushed the MRESOLVER-278-onSessionClose-rework branch from f31ce42 to e1575d1 Compare October 28, 2022 14:27
Key changes:
* It is not session to register handlers against, but RepositorySystem
* DefaultRepositorySystemSession class deprecated, but will work as intended with a
  slight semantic change (it's handlers are executed on repo system shutdown)
* introduced MutableRepositorySystemSession and used instead, session creation
  should be done using RepositorySystem and handle it as a resource
* locking: partially undo PR apache#196, but do not use System properties but
  container injecter parameters instead (works with SISU only!)
@cstamas cstamas force-pushed the MRESOLVER-278-onSessionClose-rework branch from c3ccecd to a8a550c Compare October 28, 2022 16:38
@cstamas cstamas merged commit 0572277 into apache:master Oct 28, 2022
@cstamas cstamas deleted the MRESOLVER-278-onSessionClose-rework branch October 28, 2022 20:24
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.

1 participant