[GitHub] jmeter issue #233: Bug 60274

2018-06-15 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/233 It appears similar feature is now available a 3rd party plugin : * https://jmeter-plugins.org/?search=Weighted So I propose to close this one. ---

[GitHub] jmeter issue #233: Bug 60274

2016-11-13 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 Agreed, I need to find some time to consider this issue. I may be that the variable resolution is not feasible without major changes. Again, thank you, Thorson On Nov

[GitHub] jmeter issue #233: Bug 60274

2016-11-13 Thread vlsi
Github user vlsi commented on the issue: https://github.com/apache/jmeter/pull/233 > I will test and consider a solution The question is if that is feasible at all to support run-time variable changes and keep the performance sane --- If your project is set up for it, you

[GitHub] jmeter issue #233: Bug 60274

2016-11-13 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 Regarding the run time variable changes, that is a really good point. I will test and consider a solution. Thank you, Thorson On Nov 12, 2016 11:39 PM, "Vladimir

[GitHub] jmeter issue #233: Bug 60274

2016-11-13 Thread vlsi
Github user vlsi commented on the issue: https://github.com/apache/jmeter/pull/233 Just in case, C and C1 has the following groovy post-processor: `vars.put("x", String.valueOf(1+Integer.parseInt(vars.get("x"` --- If your project is set up for it, you can reply to this email and

[GitHub] jmeter issue #233: Bug 60274

2016-11-13 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 That is strange about the nesting not working, given that I manually tested nesting, and it is one of the unit tests as well. I will investigate. On Nov 13, 2016 12:08 AM, "Vladimir

[GitHub] jmeter issue #233: Bug 60274

2016-11-13 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 The reason that I do not rely on the seed is because, as I understand it, you can count on seed to be consistent within am implementation of random, but not across implementations.

[GitHub] jmeter issue #233: Bug 60274

2016-11-13 Thread vlsi
Github user vlsi commented on the issue: https://github.com/apache/jmeter/pull/233 While being able to enable-disable and rename the children from WDC UI is nice, I wonder how many worms are hidden in that "keep WDC and children tree" consistent can. Nested "Weighted

[GitHub] jmeter issue #233: Bug 60274

2016-11-12 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 Not yet, things got hectic for me at work, will resolve in a little while On Nov 12, 2016 5:38 AM, "Philippe M" wrote: > Hi @blindio

[GitHub] jmeter issue #233: Bug 60274

2016-11-12 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/233 Hi @blindio , Did you have time working on the reported issues ? Thanks --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] jmeter issue #233: Bug 60274

2016-10-31 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/233 Hi Again, For info, in your commit, no need to include the HTML files, only XML for documentation. Thanks --- If your project is set up for it, you can reply to this email and have your

[GitHub] jmeter issue #233: Bug 60274

2016-10-31 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 Thank you Philippe, I will make these changes in the next day or so --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project

[GitHub] jmeter issue #233: Bug 60274

2016-10-31 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/233 Hi Thorson, Sorry for late reply, we are in the middle of 3.1 release preparation and I was on holiday. Thanks again for you PR. I started reviewing, few notes: -

[GitHub] jmeter issue #233: Bug 60274

2016-10-31 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 Hi Philippe- I have not heard anything for a while and wanted to know if you had an idea when someone would be able to review this. Thanks, Thorson --- If your project is

[GitHub] jmeter issue #233: Bug 60274

2016-10-21 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 I look forward to your feedback --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] jmeter issue #233: Bug 60274

2016-10-19 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/233 Thanks for this contribution ! We will review it ASAP and give you feedback. Regards Philippe M. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] jmeter issue #233: Bug 60274

2016-10-18 Thread blindio
Github user blindio commented on the issue: https://github.com/apache/jmeter/pull/233 All build and test errors that I believe that I introduced are resolved. Please tell me if this is incorrect. --- If your project is set up for it, you can reply to this email and have your reply