lburgazzoli commented on a change in pull request #1344:
URL: https://github.com/apache/camel-quarkus/pull/1344#discussion_r440027426



##########
File path: examples/timer-log-cdi/src/main/resources/application.properties
##########
@@ -14,33 +14,17 @@
 ## See the License for the specific language governing permissions and
 ## limitations under the License.
 ## ---------------------------------------------------------------------------
+
 #
 # Quarkus
 #
 quarkus.banner.enabled = false
 
-#
-# Quarkus - Camel
-#
-
-# disable build time route discovery as the only
-# route defined in this project is based on CDI.
-#
-# This is not strictly needed for the correctness
-# because camel-quarkus automatically
-# filters auto-discovered routes that targets CDI
-# but if is is known that routes are all leveraging
-# CDI, this option will give some little boost to
-# the build process
-quarkus.camel.main.routes-discovery.enabled = false
-
-#
-# Camel
-#
-camel.context.name = quarkus-camel-example-timer-log-cdi
+# camel look-up beans using BeanManager so we don't want
+# ArC to remove beans without injection points.
+quarkus.arc.remove-unused-beans = false

Review comment:
       It is possible to use the `@Unremovable` annotation as alternative but 
it's not better.
   
   We probably need to discuss this topic with the Quarkus team and see if 
through a build item it could be possible to reverse the behaviour so not to 
remove beans unless that feature is turned on explicitly.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to