First shot at integrating mp-metrics. #3
Conversation
No clue if the interceptors are intercepting.
| metrics.createRegistries(); | ||
| } | ||
|
|
||
| try (BytecodeRecorder recorder = processorContext.addDeploymentTask(RuntimePriority.WELD_DEPLOYMENT + 30 ) ) { |
There was a problem hiding this comment.
I dunno, could this be moved to static instead of deployment?
There was a problem hiding this comment.
Seems like the answer is "yes"
| } | ||
|
|
||
| public void addInterceptor(SeContainerInitializer initialize, Class<?> interceptorClass) { | ||
| initialize.enableInterceptors(interceptorClass); |
There was a problem hiding this comment.
SeContainerInitializer.enableInterceptors() only enables the interceptors for the synthetic bean archive. MetricsInterceptor and friends are enabled globally (for the application) using @Priority - so there's no need to enable them locally. In fact, this enablement is just ignored.
|
Hm, so because the discovery is disabled completely the smallrye extension is not used and you're trying to simulate the functionality, right? Of course, you could use |
Make BeanContainer.RequestAction more generic
4 participants
No clue if the interceptors are intercepting.