adamsaghy commented on code in PR #2891: URL: https://github.com/apache/fineract/pull/2891#discussion_r1072171238
########## fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/messaging/jms/ActiveMQMessageFactory.java: ########## @@ -16,21 +16,20 @@ * specific language governing permissions and limitations * under the License. */ -package org.apache.fineract.infrastructure.event.external.config; +package org.apache.fineract.infrastructure.core.messaging.jms; -import org.springframework.boot.autoconfigure.condition.ConditionalOnProperty; -import org.springframework.context.annotation.Bean; -import org.springframework.context.annotation.Configuration; -import org.springframework.integration.channel.DirectChannel; -import org.springframework.integration.config.EnableIntegration; +import javax.jms.BytesMessage; +import javax.jms.JMSException; +import org.apache.activemq.command.ActiveMQBytesMessage; +import org.springframework.stereotype.Component; -@Configuration -@EnableIntegration -@ConditionalOnProperty(value = "fineract.events.external.enabled", havingValue = "true") -public class ExternalEventProducerConfiguration { +@Component Review Comment: Dont we need condition whether we wanna have ActiveMQMessageFactory? Or will it be the de facto implementation and others might override it only? -- 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. To unsubscribe, e-mail: commits-unsubscr...@fineract.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org