diff options
author | Daniel Stenberg <daniel@haxx.se> | 2017-11-04 12:56:30 +0100 |
---|---|---|
committer | Daniel Stenberg <daniel@haxx.se> | 2017-11-04 20:56:26 +0100 |
commit | ee8016b3de0b0e3e9a352a1123c5e6272848aa55 (patch) | |
tree | 65362606421074a0e5fc9582175afba34e47e5a6 /tests | |
parent | 90abb74ff0e3134d8647722cee36b1815c14143d (diff) | |
download | curl-ee8016b3de0b0e3e9a352a1123c5e6272848aa55.tar.gz |
curl: speed up handling of many URLs
By properly keeping track of the last entry in the list of URLs/uploads
to handle, curl now avoids many meaningless traverses of the list which
speeds up many-URL handling *MASSIVELY* (several magnitudes on 100K
URLs).
Added test 1291, to verify that it doesn't take ages - but we don't have
any detection of "too slow" command in the test suite.
Reported-by: arainchik on github
Fixes #1959
Closes #2052
Diffstat (limited to 'tests')
-rw-r--r-- | tests/data/Makefile.inc | 2 | ||||
-rw-r--r-- | tests/data/test1291 | 51 |
2 files changed, 52 insertions, 1 deletions
diff --git a/tests/data/Makefile.inc b/tests/data/Makefile.inc index 35c41a5c1..9104f34f5 100644 --- a/tests/data/Makefile.inc +++ b/tests/data/Makefile.inc @@ -137,7 +137,7 @@ test1252 test1253 test1254 test1255 test1256 test1257 test1258 test1259 \ test1260 test1261 test1262 \ \ test1280 test1281 test1282 test1283 test1284 test1285 test1286 test1287 \ -test1288 test1289 test1290 \ +test1288 test1289 test1290 test1291 \ test1298 test1299 \ test1300 test1301 test1302 test1303 test1304 test1305 test1306 test1307 \ test1308 test1309 test1310 test1311 test1312 test1313 test1314 test1315 \ diff --git a/tests/data/test1291 b/tests/data/test1291 new file mode 100644 index 000000000..12d65f3d8 --- /dev/null +++ b/tests/data/test1291 @@ -0,0 +1,51 @@ +# This test case is primarily meant to verify that parsing and adding the 100K +# files is a swift operation. +# +<testcase> +<info> +<keywords> +HTTP +HTTP PUT +</keywords> +</info> + +# +# Server-side +<reply> +<data> +</data> +</reply> + +# Client-side +<client> +<server> +none +</server> +<name> +Attempt to upload 100K files but fail immediately +</name> +<command> +-K log/cmd1291 --fail-early +</command> +<file name="log/upload-this"> +XXXXXXXx +</file> +# generate the config file +<precheck> +perl -e 'for(1 .. 100000) { printf("upload-file=log/upload-this\nurl=htttttp://non-existing-host.haxx.se/upload/1291\n", $_);}' > log/cmd1291; +</precheck> +</client> + +# Verify data after the test has been "shot" +<verify> +<errorcode> +1 +</errorcode> + +# we disable valgrind here since it takes 40+ seconds even on a fairly snappy +# machine +<valgrind> +disable +</valgrind> +</verify> +</testcase> |