[infinispan-issues] [JBoss JIRA] Resolved: (ISPN-1063) Validation in TransactionFactory.init should be moved to ConfigurationValidatingVisitor

[Mircea Markus (JIRA)]

     [ https://issues.jboss.org/browse/ISPN-1063?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Mircea Markus resolved ISPN-1063.
---------------------------------

    Resolution: Won't Fix


Thinking some more, in TransctionFactory.init I'll have to walk through all the valid combinations anyway, even if the validation would be placed in the visitor. I generally agree that the validations should be held there - but for this particular scenario I think the code is better the way it is.

> Validation in TransactionFactory.init should be moved to ConfigurationValidatingVisitor
> ---------------------------------------------------------------------------------------
>
>                 Key: ISPN-1063
>                 URL: https://issues.jboss.org/browse/ISPN-1063
>             Project: Infinispan
>          Issue Type: Enhancement
>          Components: Configuration, Transactions
>            Reporter: Galder Zamarreño
>            Assignee: Mircea Markus
>             Fix For: 5.0.0.CR2
>
>
> 2:25:00 PM galderz: mmarkus: hi
> 2:25:26 PM mmarkus: hi galderz
> 2:26:11 PM galderz: hey mmarkus, i was wondering if the validation in TransactionFactory.init would be better suited for ConfigurationValidatingVisitor ?
> 2:26:30 PM galderz: i mean, the validation about the incompatible configuration options
> 2:28:49 PM mmarkus: galderz: which validation exactly? there's no validation in TransactionFactory.init ..
> 2:33:51 PM galderz:       if (txFactoryEnum == null) { ....
> 2:34:01 PM galderz: Unsupported combination (dldEnabled, recoveryEnabled, xa) = (%s, %s, %s)
> 2:34:14 PM galderz: that's validation of the configuration afact
> 2:34:28 PM galderz:       boolean dldEnabled = configuration.isEnableDeadlockDetection();
>       boolean recoveryEnabled = configuration.isTransactionRecoveryEnabled();
>       boolean xa = !configuration.isUseSynchronizationForTransactions();
> 2:37:58 PM mmarkus: galderz: yes, good point it should move to ConfigurationValidatingVisitor

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira