HSEARCH-4330 Configurable mapping for entities added by the database-polling coordination strategy #2833
Conversation
yrodiere
force-pushed
the
HSEARCH-4330
branch
from
6e59a68 to
be79db4
Compare
| ServiceRegistry serviceRegistry = metadata.getMetadataBuildingOptions().getServiceRegistry(); | ||
| ConfigurationService service = HibernateOrmSpiUtils.serviceOrFail( | ||
| serviceRegistry, ConfigurationService.class ); | ||
| String entityDefinition = AGENT_ENTITY_MAPPING.get( | ||
| AllAwareConfigurationPropertySource.fromMap( (Map<String, ?>) service.getSettings() ) ); |
There was a problem hiding this comment.
If you need to pass additional parameters (like the configuration source) to the mapping producers, it would probably be better to give up on implementing AdditionalJaxbMappingProducer and use a custom interface. I'll push an additional commit.
| ServiceRegistry serviceRegistry = metadata.getMetadataBuildingOptions().getServiceRegistry(); | ||
| ConfigurationService service = HibernateOrmSpiUtils.serviceOrFail( | ||
| serviceRegistry, ConfigurationService.class ); | ||
| String entityDefinition = OUTBOXEVENT_ENTITY_MAPPING.get( | ||
| AllAwareConfigurationPropertySource.fromMap( (Map<String, ?>) service.getSettings() ) ); |
There was a problem hiding this comment.
Same here. I'll push an additional commit.
| import org.hibernate.service.Service; | ||
| import org.hibernate.service.ServiceRegistry; | ||
|
|
||
| public final class HibernateOrmSpiUtils { |
There was a problem hiding this comment.
This should disappear with my additional commit.
…ion property source as an argument
yrodiere
force-pushed
the
HSEARCH-4330
branch
from
be79db4 to
eaefbb1
Compare
|
Looks good, thanks. As I said above, I pushed an additional commit to simplify how the configuration property source is accessed. Let's merge once CI agrees. |
|
Kudos, SonarCloud Quality Gate passed!
|
2 participants
https://hibernate.atlassian.net/browse/HSEARCH-4330