RE: [OpenIV:0551] Re: Bugzilla Enhancement: "Auto-Correlation as core functionality in JMeter"

2019-12-17 Thread Rahul Mehta
Hi Imoto, Thanks for considering, below are my comments. On Tuesday, December 17, 2019, Takashi Imoto wrote: > To: Rahul-san. > > > > Thank you for comments. > > > > > Why you are manually selecting the corealtion candidates? it should be > automated. > > > > Auto correlation feature list up

Re: Hardcoded setEnableAutoCommitOnReturn=false

2019-12-17 Thread Philippe Mouawad
My proposal was in case we don't add ability to configure it. So ok by me with Vladimir's proposal although it should be documented that settings are kind of related. On Tue, Dec 17, 2019 at 10:45 PM Dmitry Sherstobitov wrote: > Philippe, > > I don't think this will be the correct solution.

Re: Hardcoded setEnableAutoCommitOnReturn=false

2019-12-17 Thread Dmitry Sherstobitov
Philippe, I don't think this will be the correct solution. It's still hardcoded in this case and strictly depends on autoCommit which is not obvious. I agree with Vladimir - this should be configurable. On Wed, Dec 18, 2019 at 12:07 AM Vladimir Sitnikov < sitnikov.vladi...@gmail.com> wrote: >

Re: Hardcoded setEnableAutoCommitOnReturn=false

2019-12-17 Thread Vladimir Sitnikov
Philippe>dataSource.setEnableAutoCommitOnReturn(isAutocommit()); What do you think of adding a checkbox like "rollback on returning connections to the pool"? Dmitry>I've created issue. Currently I can't find solution for that without Dmitry>changing source code :( I guess it would make sense to

Re: Hardcoded setEnableAutoCommitOnReturn=false

2019-12-17 Thread Philippe Mouawad
Hello, I guess correct fix would be: dataSource.setEnableAutoCommitOnReturn(isAutocommit()); Can you try with this code ? Thank you On Tue, Dec 17, 2019 at 9:45 PM Dmitry Sherstobitov wrote: > Vladimir, > > I've created issue. Currently I can't find solution for that without > changing

Re: Hardcoded setEnableAutoCommitOnReturn=false

2019-12-17 Thread Dmitry Sherstobitov
Vladimir, I've created issue. Currently I can't find solution for that without changing source code :( https://bz.apache.org/bugzilla/show_bug.cgi?id=64012 On Tue, Dec 17, 2019 at 12:27 AM Dmitry Sherstobitov wrote: > About cause - it's true. I chose 3.3 version because it's first version >