Bug #69186 | Wasted work in method test_rewind | ||
---|---|---|---|
Submitted: | 9 May 2013 15:47 | Modified: | 21 Jun 2013 6:45 |
Reporter: | Po-Chun Chang (OCA) | Email Updates: | |
Status: | Not a Bug | Impact on me: | |
Category: | MySQL Server: DML | Severity: | S5 (Performance) |
Version: | 5.6, 5.7 | OS: | Any |
Assigned to: | CPU Architecture: | Any | |
Tags: | Contribution, patch, performance |
[9 May 2013 15:47]
Po-Chun Chang
[9 May 2013 15:48]
Po-Chun Chang
Suggested patch
Attachment: patch.diff (text/plain), 461 bytes.
[9 May 2013 15:48]
Po-Chun Chang
Suggested patch
Attachment: patch2.diff (text/plain), 515 bytes.
[9 May 2013 15:48]
Po-Chun Chang
Suggested patch
Attachment: patch3.diff (text/plain), 648 bytes.
[9 May 2013 15:48]
Po-Chun Chang
Suggested patch
Attachment: patch4.diff (text/plain), 537 bytes.
[9 May 2013 17:43]
MySQL Verification Team
Please submit the OCA. Thanks.
[9 May 2013 23:33]
Po-Chun Chang
OCA remitted, thanks.
[14 Jun 2013 10:19]
Anitha Gopi
Thanks for the contribution. Can you please upload the patch in the contributions section
[15 Jun 2013 18:24]
Po-Chun Chang
Suggested patch (*) I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.
Contribution: patch.diff (application/octet-stream, text), 461 bytes.
[15 Jun 2013 18:24]
Po-Chun Chang
Suggested patch (*) I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.
Contribution: patch2.diff (application/octet-stream, text), 515 bytes.
[15 Jun 2013 18:24]
Po-Chun Chang
Suggested patch (*) I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.
Contribution: patch3.diff (application/octet-stream, text), 648 bytes.
[15 Jun 2013 18:25]
Po-Chun Chang
Suggested patch (*) I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it.
Contribution: patch4.diff (application/octet-stream, text), 537 bytes.
[21 Jun 2013 6:44]
Nirbhay Choubey
IMHO the reported bug and hence the patch is invalid. See, 'silent' option is normally to reduce the verbosity of a test but in itself should not cut-down some important (execution) steps of the test case. For example, if we look at original test_rewind : .. /* retreive all result sets till we are at the end */ while(!mysql_stmt_fetch(stmt)) if (!opt_silent) printf("fetched result:%ld\n", Data); DIE_UNLESS(rc != MYSQL_NO_DATA); .. The test tries to fetch all the rows and later asserts that the final return should result in MYSQL_NO_DATA. With the supplied patch, the test would fail with : check failed: 'rc == MYSQL_NO_DATA' Same applies to other contributed patches for test_bug9478, test_bug11172.
[25 Jun 2013 11:02]
Ståle Deraas
Hi Po-Chun Chang, Your contributions were rejected due to the explanation given above by Nirbhay. But thank you anyway! Staale