-
Notifications
You must be signed in to change notification settings - Fork 304
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
FISH-6430: skipping BdId TransactionalExtension type to not generate error logs #5963
FISH-6430: skipping BdId TransactionalExtension type to not generate error logs #5963
Conversation
Jenkins test please |
...er/web/weld-integration/src/main/java/org/glassfish/weld/services/InjectionServicesImpl.java
Outdated
Show resolved
Hide resolved
Jenkins test please |
Jenkins test please |
Jenkins test please |
Jenkins test please |
3 similar comments
Jenkins test please |
Jenkins test please |
Jenkins test please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The reproducer attached to the issue doesn't appear to work?
...er/web/weld-integration/src/main/java/org/glassfish/weld/services/InjectionServicesImpl.java
Outdated
Show resolved
Hide resolved
...er/web/weld-integration/src/main/java/org/glassfish/weld/services/InjectionServicesImpl.java
Outdated
Show resolved
Hide resolved
Jenkins test please |
2 similar comments
Jenkins test please |
Jenkins test please |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Automating the reproducer as a test would be nice, though I appreciate scraping logs for the ommision of a FINE level log isn't exactly a "precise" test, so approving as-is.
Description
This is a fix to prevent error logs related to inject TransactionalScopedCDIEventHelperImpl for TransactionScoped CDI Beans
Important Info
Blockers
Testing
New tests
Testing Performed
local testing using the attached reproducer (adjusted to be compatible with most recent version of jakarta) from the ticket: FISH-6430
logs looks more cleaned:
`[#|2022-09-27T12:01:10.483-0500|INFO|Payara 6.2022.1.Alpha5-SNAPSHOT|javax.enterprise.system.core|_ThreadID=145;_ThreadName=admin-thread-pool::admin-listener(1);_TimeMillis=1664298070483;_LevelValue=800;|
simple-ear was successfully deployed in 3,935 milliseconds.|#]
[#|2022-09-27T12:02:04.044-0500|INFO|Payara 6.2022.1.Alpha5-SNAPSHOT||_ThreadID=131;_ThreadName=http-thread-pool::http-listener-1(3);_TimeMillis=1664298124044;_LevelValue=800;|
ServiceBean::doWork() called|#]
[#|2022-09-27T12:02:10.350-0500|INFO|Payara 6.2022.1.Alpha5-SNAPSHOT||_ThreadID=131;_ThreadName=http-thread-pool::http-listener-1(3);_TimeMillis=1664298130350;_LevelValue=800;|
TestTransactionalScopeBean initialized. Value is 1664298130350|#]
[#|2022-09-27T12:02:59.559-0500|INFO|Payara 6.2022.1.Alpha5-SNAPSHOT||_ThreadID=131;_ThreadName=http-thread-pool::http-listener-1(3);_TimeMillis=1664298179559;_LevelValue=800;|
ServiceBean:: testTransactionalScope value is 1664298130350|#]
[#|2022-09-27T12:03:17.772-0500|INFO|Payara 6.2022.1.Alpha5-SNAPSHOT||_ThreadID=131;_ThreadName=http-thread-pool::http-listener-1(3);_TimeMillis=1664298197772;_LevelValue=800;|
NestedServiceBean::doWork() called|#]
[#|2022-09-27T12:03:17.773-0500|INFO|Payara 6.2022.1.Alpha5-SNAPSHOT||_ThreadID=131;_ThreadName=http-thread-pool::http-listener-1(3);_TimeMillis=1664298197773;_LevelValue=800;|
NestedServiceBean:: testTransactionalScope value is 1664298130350|#]
[#|2022-09-27T12:04:24.385-0500|WARNING|Payara 6.2022.1.Alpha5-SNAPSHOT|com.hazelcast.internal.cluster.impl.ClusterHeartbeatManager|_ThreadID=228;_ThreadName=hz.suspicious_bouman.cached.thread-29;_TimeMillis=1664298264385;_LevelValue=900;|
[172.21.41.173]:4900 [development] [5.1.1] Resetting heartbeat timestamps because of huge system clock jump! Clock-Jump: 43667 ms, Heartbeat-Timeout: 60000 ms|#]
[#|2022-09-27T12:04:26.968-0500|INFO|Payara 6.2022.1.Alpha5-SNAPSHOT||_ThreadID=131;_ThreadName=http-thread-pool::http-listener-1(3);_TimeMillis=1664298266968;_LevelValue=800;|
TestTransactionalScopeBean destroyed. Value is 1664298130350|#]`
Testing Environment
Ubuntu 20.04, Azul JDK 11, Maven 3.8.6
Documentation
Notes for Reviewers