Bug #59146 | nist.nist_all fails: 1.uses deprecated syntax; 2.test result needs to be updated | ||
---|---|---|---|
Submitted: | 24 Dec 2010 7:14 | Modified: | 25 Oct 2018 5:46 |
Reporter: | Alexander Nozdrin | Email Updates: | |
Status: | Can't repeat | Impact on me: | |
Category: | Tests: Server | Severity: | S3 (Non-critical) |
Version: | 5.6.1,mysql-trunk | OS: | Any |
Assigned to: | Bernt Marius Johnsen | CPU Architecture: | Any |
Tags: | pb2, test failure |
[24 Dec 2010 7:14]
Alexander Nozdrin
[27 Dec 2010 18:24]
Magne Mæhre
The "bug" is partly a consequence of WL#5185, and reflects a wanted change. The warnings about "'@@storage_engine' is deprecated and will be removed in a future release." would be best addressed by replacing every occurrence of 'storage_engine' with 'default_storage_engine' in the test case. The variable 'storage_engine' will be removed in a soon-to-come release.
[28 Dec 2010 4:30]
Anitha Gopi
nist suite should be included in the daily/weekly runs for earlier detection of issues
[28 Dec 2010 4:43]
Anitha Gopi
Also see WL#5265 * WL#4738: Deprecate the storage_engine system variable. Reason: WL#4738 added default_storage_engine, which has the same meaning, but the name matches the --default-storage-engine command-line option more closely. storage_engine does not have the same obvious correspondence to --default-storage-engine.
[29 Dec 2010 4:11]
Anitha Gopi
After changing storage_engine to default storage engine the test still fails with the following error +++ /home/anitha/mysql/build/mysql-trunk-bugfixing/usr/local/mysql/mysql-test/suite/nist/r/nist_all.reject 2010-12-29 07:05:36.310741622 +0300 @@ -981,7 +981,7 @@ TEST-INFO: passed GRANT SELECT ON CUGINI.BADG2 TO SCHANZLE@localhost; -ERROR 42000: SELECT,GRANT command denied to user 'FLATER'@'localhost' for table 'BADG2' +ERROR 42000: SELECT, GRANT command denied to user 'FLATER'@'localhost' for table 'BADG2' TEST-INFO: passed @@ -11943,7 +11943,7 @@ E3 13 Vienna COMMIT WORK; DROP VIEW STAFF CASCADE; -ERROR 42S02: Unknown table 'STAFF' +ERROR 42S02: Unknown table 'FLATER.STAFF' TEST-INFO: The last statement failed, because MySQL does not - understand the syntax or mysqltest: Result content mismatch
[29 Dec 2010 5:29]
Anitha Gopi
http://dev.mysql.com/doc/refman/5.5/en/news-5-5-3.html The storage_engine system variable is deprecated in favor of the new system variable default_storage_engine. This enables pairing of the --default-storage-engine command-line option with a system variable of a more closely corresponding name.
[29 Dec 2010 5:51]
Anitha Gopi
The difference in error message is not seen in 5.5. The changes needed to fix the test will make it incompatible with 5.1 and 5.5. This test is maintained in mysql-test-extra and that makes it hard to handle this. We either have to create separate folders for the different versions in mysql-test-extra or move nist suite under mysql code tree. Do we have any particular reason to keep it out of mysql
[29 Dec 2010 5:54]
Anitha Gopi
Patch for trunk
Attachment: BUG59146.txt (text/plain), 5.04 KiB.
[3 Jan 2011 13:50]
Matthias Leich
Complete patch for trunk
Attachment: BUG59146.txt (text/plain), 6.28 KiB.
[3 Jan 2011 14:26]
Matthias Leich
Hi Anitha, your patch fixes the reported problem. But in order to ensure the consistency of the test suite some other files have to be patched in addition. mysql-test/suite/nist/r/nist_data.result Some modified message from test schema5 was missing. mysql-test/suite/nist/r/schema/schema5.result mysql-test/suite/nist/r/sql/dml114.result Again some modified messages were missing. From the logics; nist_data.result and nist_all.result are basically concatenations of the results of the performed sub tests (e.g. schema5, dml114, ....). Maintaining result snipplets like dml114.result allows to easy remove sub tests from nist_all temporarily without loosing the already checked expected results. I have uploaded a BUG59146.txt which contains the additional patches. ready to push