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

Annotation transformer - add TRACE logging #29409

Merged
merged 1 commit into from Nov 23, 2022
Merged

Annotation transformer - add TRACE logging #29409

merged 1 commit into from Nov 23, 2022

Conversation

mkouba
Copy link
Contributor

@mkouba mkouba commented Nov 22, 2022

  • so that it's possible to find out which transformer performed a transformation of a particular target

@mkouba mkouba requested a review from manovotn November 22, 2022 13:53
@quarkus-bot quarkus-bot bot added the area/arc Issue related to ARC (dependency injection) label Nov 22, 2022
@mkouba
Copy link
Contributor Author

mkouba commented Nov 22, 2022

The output looks like:

2022-11-22 15:15:01,321 TRACE [io.qua.arc.pro.AnnotationStore$TransformationContextImpl] (build-16) Transforming annotations of rest.Application 
	io.quarkus.arc.deployment.AutoAddScopeProcessor$1.transform(AutoAddScopeProcessor.java:98)
	io.quarkus.arc.processor.AnnotationStore.transform(AnnotationStore.java:108)
	java.base/java.util.concurrent.ConcurrentHashMap.computeIfAbsent(ConcurrentHashMap.java:1705)
	io.quarkus.arc.processor.AnnotationStore.getAnnotations(AnnotationStore.java:61)
	io.quarkus.arc.processor.AnnotationStore.hasAnnotation(AnnotationStore.java:85)
	io.quarkus.arc.processor.BeanDeployment.findBeans(BeanDeployment.java:893)
	io.quarkus.arc.processor.BeanDeployment.registerBeans(BeanDeployment.java:252)
	...
2022-11-22 15:15:01,321 TRACE [io.qua.arc.pro.AnnotationsTransformationContext] (build-16) Annotations of rest.Application transformed: [@javax.enterprise.context.RequestScoped]

@mkouba mkouba force-pushed the annotation-trasformer-trace-logging branch from 5fa402c to b603f0b Compare November 22, 2022 14:19
@mkouba
Annotation transformer - add TRACE logging
624eb45 
- so that it's possible to find out which transformer performed a
transformation of a particular target
@mkouba mkouba force-pushed the annotation-trasformer-trace-logging branch from b603f0b to 624eb45 Compare November 22, 2022 14:19
@quarkus-bot quarkus-bot bot added this to To do in Quarkus Documentation Nov 22, 2022
Quarkus Documentation automation moved this from To do to Reviewer approved Nov 22, 2022
@quarkus-bot

This comment has been minimized.

Sign in to view
manovotn
manovotn approved these changes Nov 22, 2022
View reviewed changes
Copy link
Contributor

@manovotn manovotn left a comment

Choose a reason for hiding this comment

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

Great idea!

@manovotn manovotn added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Nov 22, 2022
@quarkus-bot

This comment has been minimized.

Sign in to view
@mkouba mkouba merged commit 3101519 into quarkusio:main Nov 23, 2022
Quarkus Documentation automation moved this from Reviewer approved to Done Nov 23, 2022
@quarkus-bot quarkus-bot bot added this to the 2.15 - main milestone Nov 23, 2022
@quarkus-bot quarkus-bot bot removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Nov 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Ladicek Ladicek Ladicek approved these changes

@manovotn manovotn manovotn approved these changes

Assignees
No one assigned
Labels
area/arc Issue related to ARC (dependency injection) area/documentation
Projects
Quarkus Documentation
Done
Milestone
2.15.0.CR1
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@mkouba @Ladicek @manovotn