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

Update EJB and Web JaccService to not be immediate=true #30629

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Update EJB and Web JaccService to not be immediate=true #30629

merged 1 commit into from
Jan 24, 2025
+3 −3

Conversation

jhanders34
Copy link
Member

  • When restructuring the JaccServiceImpl code the EJB and Web versions were made immediate=true when the original JaccServiceImpl was not which causes some side effects where things were initialized in the wrong order. This change fixes it.

This PR fixes a regression caused by the changes in PR #30293.

@jhanders34
Update EJB and Web JaccService to not be immediate=true
eb71cfd 
- When restructuring the JaccServiceImpl code the EJB and Web versions
were made immediate=true when the original JaccServiceImpl was not which
causes some side effects where things were initialized in the wrong
order.  This change fixes it.
@jhanders34 jhanders34 requested a review from ayoho January 23, 2025 21:29
@jhanders34 jhanders34 self-assigned this Jan 23, 2025
@jhanders34
Copy link
Member Author

jhanders34 commented Jan 23, 2025 •
edited by LibbyBot
Loading

!build (view Open Liberty Personal Build - ✅ completed successfully!)

Note: Target locations of links might be accessible only to IBM employees.

@jhanders34
Copy link
Member Author

jhanders34 commented Jan 23, 2025 •
edited by LibbyBot
Loading

!build
(view Open Liberty Personal Build - ✅ completed successfully!)
global.java2.sec=true

Note: Target locations of links might be accessible only to IBM employees.

@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 2 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

1 similar comment
@LibbyBot
Copy link

Code analysis and actions

DO NOT DELETE THIS COMMENT.
  • 2 product code files were changed.
  • Please describe in a separate comment how you tested your changes.

@jhanders34 jhanders34 merged commit fea7053 into OpenLiberty:integration Jan 24, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ayoho ayoho ayoho approved these changes

Assignees

@jhanders34 jhanders34

Labels
CLA Signed team:Performance
Projects
None yet
Milestone
[Iteration 25.2] Jan 20 - Jan 31
Development

Successfully merging this pull request may close these issues.
3 participants
@jhanders34 @LibbyBot @ayoho