[jira] [Commented] (DELTASPIKE-1238) Make EnvironmentAwareTransactionStrategy the default

2017-03-07 Thread Mark Struberg (JIRA)

[ 
https://issues.apache.org/jira/browse/DELTASPIKE-1238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15898948#comment-15898948
 ] 

Mark Struberg commented on DELTASPIKE-1238:
---

John, is this just because of the broken BDA handling in Weld? 
If so can you please try to enable the alternative via AfterTypeDiscovery?
Or use DeltaSpike global-alternatives.
I'm also against cannibalising performance for fixing a broken setup.

> Make EnvironmentAwareTransactionStrategy the default
> 
>
> Key: DELTASPIKE-1238
> URL: https://issues.apache.org/jira/browse/DELTASPIKE-1238
> Project: DeltaSpike
>  Issue Type: Improvement
>Reporter: John D. Ament
>Assignee: John D. Ament
>
> Make EnvironmentAwareTransactionStrategy the default transaction strategy, 
> since it properly looks up container managed, resource local without 
> requiring the user to do anything.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (DELTASPIKE-1238) Make EnvironmentAwareTransactionStrategy the default

2017-03-06 Thread Romain Manni-Bucau (JIRA)

[ 
https://issues.apache.org/jira/browse/DELTASPIKE-1238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15897656#comment-15897656
 ] 

Romain Manni-Bucau commented on DELTASPIKE-1238:


-1 in current form, the jta detection is super slow cause of the exception, if 
we integrate it smoother with other parts like JPA module which has the info 
then we can probably revise this but today it is not an option at runtime I 
fear.

> Make EnvironmentAwareTransactionStrategy the default
> 
>
> Key: DELTASPIKE-1238
> URL: https://issues.apache.org/jira/browse/DELTASPIKE-1238
> Project: DeltaSpike
>  Issue Type: Improvement
>Reporter: John D. Ament
>Assignee: John D. Ament
>
> Make EnvironmentAwareTransactionStrategy the default transaction strategy, 
> since it properly looks up container managed, resource local without 
> requiring the user to do anything.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (DELTASPIKE-1238) Make EnvironmentAwareTransactionStrategy the default

2017-03-06 Thread John D. Ament (JIRA)

[ 
https://issues.apache.org/jira/browse/DELTASPIKE-1238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15897654#comment-15897654
 ] 

John D. Ament commented on DELTASPIKE-1238:
---

Those are things that can be fixed.  We have direct feedback on the user's list 
that the default doesn't work for most use cases out of the box, which is true.

> Make EnvironmentAwareTransactionStrategy the default
> 
>
> Key: DELTASPIKE-1238
> URL: https://issues.apache.org/jira/browse/DELTASPIKE-1238
> Project: DeltaSpike
>  Issue Type: Improvement
>Reporter: John D. Ament
>Assignee: John D. Ament
>
> Make EnvironmentAwareTransactionStrategy the default transaction strategy, 
> since it properly looks up container managed, resource local without 
> requiring the user to do anything.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)


[jira] [Commented] (DELTASPIKE-1238) Make EnvironmentAwareTransactionStrategy the default

2017-03-06 Thread Mark Struberg (JIRA)

[ 
https://issues.apache.org/jira/browse/DELTASPIKE-1238?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=15897221#comment-15897221
 ] 

Mark Struberg commented on DELTASPIKE-1238:
---

Not quite sure. The EnvironmentAwareTxStrategy comes with quite a performance 
impact.
It's very easy to enable it anyway, so why bother?
So from a pure backward compat view I'd leave it as is.
Projects who use EnvironmentAwareTxStrategy will have the alternative anyway 
already.

> Make EnvironmentAwareTransactionStrategy the default
> 
>
> Key: DELTASPIKE-1238
> URL: https://issues.apache.org/jira/browse/DELTASPIKE-1238
> Project: DeltaSpike
>  Issue Type: Improvement
>Reporter: John D. Ament
>Assignee: John D. Ament
>
> Make EnvironmentAwareTransactionStrategy the default transaction strategy, 
> since it properly looks up container managed, resource local without 
> requiring the user to do anything.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)