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.

Sign up for GitHub

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

Jump to bottom

WFLY-7628 Distributed EJB timers #15053

Merged
merged 51 commits into from Jul 20, 2022
Merged

WFLY-7628 Distributed EJB timers #15053

merged 51 commits into from Jul 20, 2022

Conversation

pferraro
Copy link
Contributor

@pferraro pferraro commented Dec 24, 2021 •
edited

@pferraro
Copy link
Contributor Author

@chengfang @rhusar @rachmatowicz Please review

@github-actions github-actions bot added the deps-changed Dependencies have been checked, and there are changes highlighted in a comment label Dec 24, 2021
@pferraro pferraro added the Feature PR provides a new feature label Dec 28, 2021
@darranl darranl added the rebase-this PR has a merge conflict. label Jan 9, 2022
@darranl
Copy link
Contributor

darranl commented Jan 9, 2022

FYI this presently has a merge conflict.

@pferraro pferraro removed the rebase-this PR has a merge conflict. label Jan 14, 2022
@pferraro pferraro force-pushed the WFLY-7628 branch 3 times, most recently from 25b9c2e to 118ac7c Compare January 20, 2022 01:14
@pferraro pferraro closed this Jan 28, 2022
@pferraro pferraro reopened this Jan 28, 2022
@pferraro pferraro force-pushed the WFLY-7628 branch 6 times, most recently from bca6654 to c5befcc Compare February 8, 2022 19:12
@pferraro pferraro force-pushed the WFLY-7628 branch 3 times, most recently from bb1dd51 to b040986 Compare February 23, 2022 20:30
@pferraro pferraro force-pushed the WFLY-7628 branch 5 times, most recently from a289db1 to e434b3b Compare February 27, 2022 14:02
@pferraro pferraro force-pushed the WFLY-7628 branch 3 times, most recently from 431ce63 to cddac7d Compare March 8, 2022 20:02
@pferraro
Delete invalid test. There is no reason to expect that the TimerServi…
819eabb 
…ce management resource must still reference a canceled timer.
@pferraro
Wait a sufficiently long time rather than indefinitely.
4a405da
@pferraro
Add integration test for distributed timer service.
03ad12e
@pferraro
Improve precision of rescheduling of earlier entries.
a187254
@pferraro
Indicate reference count in TransactionBatch.toString()
c70f97d
@pferraro
Add version 2.0 of timer-service deployment descriptor namespace.
64a02fd
@pferraro
Add timer management resource to distributable-ejb subsystem.
a5b9195
@pferraro
Add default-persistent-timer-management timer-service attribute as an…
79f1771 
… alternate to default-data-store is undefined. Use new distributable timer service implementation when enabled or when specified via deployment descriptor.
@pferraro
Allow core thread timeout.
6555821
@pferraro
Add default-transient-timer-management attribute to timer-service res…
9726df4 
…ource to enable support for passivating transient timers.
@pferraro
Update DistributedTimerTestCase to enable distributed timer managemen…
9e89db1 
…t via deployment descriptor.
@pferraro
Rename "clustered timers" to "database persistent timers".
c1860a5
@pferraro
Add distributed timer docs.
8f0f98f
@pferraro
Add distributed TimerService test case that uses infinispan-server fo…
4475e8a 
…r timer persistence.
@pferraro
Javadoc fixes.
3cbc61a
@pferraro
Handle race condition if timer expired since call to TimerService#get…
4448167 
…Timers()
@pferraro
Update integration tests to validate transient timers.
c341da7
@pferraro
Update HotRodPersistentTimerServiceTestCase to configure distributed …
f967f64 
…timer service via ejb3 subsystem, while DistributedTimerServiceTestCase configures distributed timer service via deployment descriptor.
@pferraro
Update default HA profile configuration to use distributed timer serv…
a7074f1 
…ice by default.
@pferraro
ejb layer cannot be used in conjunction with ejb-dist-cache without e…
51ba7c5 
…xcluding ejb-local-cache.
@pferraro
Migrate integration tests to ManagementServerSetupTask.
6f0f223
@pferraro
Leverage constants for paths, attribute names, model keys.
0e2df61
@pferraro
Add unit test for expression resolution of max-active-timers.
b216f85
@darranl darranl mentioned this pull request Jul 18, 2022
Merged
fabiobrz
fabiobrz approved these changes Jul 20, 2022
View reviewed changes
Copy link
Contributor

@fabiobrz fabiobrz left a comment

Choose a reason for hiding this comment

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

I am approving the PR changes, specifically with respect to the tests development and community documentation.

@pferraro pferraro removed the missing-reqs Features missing one or more of the pre-merge requirements label Jul 20, 2022
@bstansberry bstansberry merged commit 9096eba into wildfly:main Jul 20, 2022
@bstansberry
Copy link
Contributor

Thanks @pferraro. It's about time! :-D

@pferraro pferraro deleted the WFLY-7628 branch April 11, 2023 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rachmatowicz rachmatowicz rachmatowicz approved these changes

@chengfang chengfang chengfang approved these changes

@fabiobrz fabiobrz fabiobrz approved these changes

Assignees
No one assigned
Labels
deps-ok Dependencies have been checked, and there are no significant changes Feature PR provides a new feature
Projects
None yet
Milestone
No milestone
6 participants
@pferraro @darranl @bstansberry @rachmatowicz @chengfang @fabiobrz