[GitHub] jmeter pull request #434: Truncate response message to avoid too large udp p...

2018-11-23 Thread mybreeze77
Github user mybreeze77 commented on a diff in the pull request: https://github.com/apache/jmeter/pull/434#discussion_r235965931 --- Diff: src/components/org/apache/jmeter/visualizers/backend/influxdb/InfluxdbBackendListenerClient.java --- @@ -196,12 +196,18 @@ private void addMetr

[GitHub] jmeter pull request #433: Bug 62935. Pass custom 'mail.*' properties to Mail...

2018-11-23 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/jmeter/pull/433 ---

[GitHub] jmeter issue #432: Bug 62870 / Templates : Add ability to provide parameters

2018-11-23 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/432 Hello Felix, I have committed the fixes following your review. Let me know if you will commit the PR (which I would prefer), otherwise I'll do it. Thanks Regards ---

[GitHub] jmeter issue #429: Fix 62887. Customized SampleLabel is ignored and overwrit...

2018-11-23 Thread pmouawad
Github user pmouawad commented on the issue: https://github.com/apache/jmeter/pull/429 @artem-fedorov , Yes if the are in the test plan. No guarantee if they are automatically handled by resources download and it is done in parallel, but since 5.0, we add the URL in the CSV to

[GitHub] jmeter issue #429: Fix 62887. Customized SampleLabel is ignored and overwrit...

2018-11-23 Thread artem-fedorov
Github user artem-fedorov commented on the issue: https://github.com/apache/jmeter/pull/429 @pmouawad In case when I have 3 resources for request url.com - url.com/res.css - url.com/res.html - url.com/res.xml Does current behaviour guaranteed that `url.com/res.css`

[GitHub] jmeter issue #433: Bug 62935. Pass custom 'mail.*' properties to Mail Reader...

2018-11-23 Thread artem-fedorov
Github user artem-fedorov commented on the issue: https://github.com/apache/jmeter/pull/433 Awesome! thanks! ---

[GitHub] jmeter pull request #434: Truncate response message to avoid too large udp p...

2018-11-23 Thread pmouawad
Github user pmouawad commented on a diff in the pull request: https://github.com/apache/jmeter/pull/434#discussion_r235892281 --- Diff: src/components/org/apache/jmeter/visualizers/backend/influxdb/InfluxdbBackendListenerClient.java --- @@ -196,12 +196,18 @@ private void addMetric

[GitHub] jmeter issue #434: Truncate response message to avoid too large udp package ...

2018-11-23 Thread codecov-io
Github user codecov-io commented on the issue: https://github.com/apache/jmeter/pull/434 # [Codecov](https://codecov.io/gh/apache/jmeter/pull/434?src=pr&el=h1) Report > Merging [#434](https://codecov.io/gh/apache/jmeter/pull/434?src=pr&el=desc) into [trunk](https://codecov.io/gh/

[GitHub] jmeter pull request #434: Truncate response message to avoid too large udp p...

2018-11-23 Thread mybreeze77
GitHub user mybreeze77 opened a pull request: https://github.com/apache/jmeter/pull/434 Truncate response message to avoid too large udp package using UdpMetricsSender In JMeter, we can use "BackendListener - InfluxdbBackendListenerClient - UdpMetricsSender" to send data as udp to