Daniel Black added a comment - 2017-01-23 04:10 from 10.2 only: https://s3.amazonaws.com/archive.travis-ci.org/jobs/193566271/log.txt --- /home/travis/build/grooverdan/mariadb-server/mysql-test/r/cte_recursive.result 2017-01-20 00:21:50.342165997 +0000 +++ /home/travis/build/grooverdan/mariadb-server/mysql-test/r/cte_recursive.reject 2017-01-20 01:05:17.674165997 +0000 @@ -2364,7 +2364,7 @@ "r_loops": 1, "rows": 2, "r_rows": 10, - "r_total_time_ms": "REPLACED"e-4, + "r_total_time_ms": "REPLACED", "filtered": 100, "r_filtered": 100, "materialized": {   mysqltest: Result length mismatch added commit to pr 292

Elena Stepanova added a comment - 2017-01-23 11:42 Yeah that's an easy fix, but I'm not quite sure that's what we need. Do we really want the exponent format in ANALYZE output to begin with? Assigning to psergey to decide.

Sergei Petrunia added a comment - 2017-01-24 11:36 I don't see an issue with exponent format, it's actually good for very big/small values. It is also nice that the patch puts the regex into one place. I've had one change request which I posted in the github PR.

Elena Stepanova added a comment - 2017-01-30 00:34 - edited Restoring previously removed comment. The failure in main.cte_recursive is mentioned above in the comments (which is why I deleted my comment earlier), but it turns out that the conversation on github suggests that it is currently considered not reproducible – incorrectly, because it does happen in buildbot: http://buildbot.askmonty.org/buildbot/builders/kvm-bintar-quantal-amd64/builds/4582/steps/test/logs/stdio main.cte_recursive w3 [ fail ] Test ended at 2017-01-29 20:16:34   CURRENT_TEST: main.cte_recursive --- /usr/local/mariadb-10.2.4-linux-x86_64/mysql-test/r/cte_recursive.result 2017-01-29 19:32:09.000000000 +0200 +++ /usr/local/mariadb-10.2.4-linux-x86_64/mysql-test/r/cte_recursive.reject 2017-01-29 20:16:34.377172490 +0200 @@ -2364,7 +2364,7 @@ "r_loops": 1, "rows": 2, "r_rows": 10, - "r_total_time_ms": "REPLACED", + "r_total_time_ms": "REPLACED"e-4, "filtered": 100, "r_filtered": 100, "materialized": {   mysqltest: Result length mismatch

Elena Stepanova added a comment - 2017-01-30 17:35 serg has fixed cte_recursive in 10.2, so you can ignore this one

Sergei Petrunia added a comment - 2017-01-30 17:36 Ok serg has fixed the 10.2 part https://github.com/MariaDB/server/commit/9f918b9cf48989a9c1fedbd3fd8849d4d6892690 . his fix is not using include/analyze-format.inc but it also works. (pull request #292 has not yet been merged from 10.1 so it's not possible to change)

Sergei Golubchik added a comment - 2017-01-30 18:03 yes, I remember that pull request, and I checked that include/analyze-format.inc wasn't in 10.2 yet. So I consider my fix being a temporary one, and on 10.1 merge it should be replaced with include/analyze-format.inc

Varun Gupta (Inactive) added a comment - 2017-11-02 07:18 Fixed with include/analyze-format.inc in 10.1.22