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"
| @@ -63,6 +63,10 @@ public void addClass(SeContainerInitializer initializer, Class<?> clazz) { | |||
| initializer.addBeanClasses(clazz); | |||
| } | |||
|
|
|||
| 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
No clue if the interceptors are intercepting.