Hi! On 20.01.2011 16:04, Sergei Golubchik wrote:
Hi, sanja!
revno: 3014 revision-id: sanja@askmonty.org-20110113180607-e5hc8zuugxxjtn6k parent: psergey@askmonty.org-20110112083018-90rw058f00uslvoz committer: sanja@askmonty.org branch nick: work-maria-5.1-CREATE timestamp: Thu 2011-01-13 20:06:07 +0200 message: New table creation/renaming block added if old encoded table present. First: there is no test case for the new (as compared with the previous
On Jan 13, sanja@askmonty.org wrote: patch) change. It just fail upgrade test which I had not noticed.
Second: I think "old_path" does not belong to the check_table_file_presence() function. The function simply checks if the file exists, there is no place for the concept of "old path" here. You need to take care of the renaming for upgrade and old path outside of check_table_file_presence().
I see 2 ways to do it: 1) Add a function for cases of renaming which will do the check (but it lead for double building old-style-path) 2) change check_table_file_presence() so that it will get old-style-path path as argument and build it for both checks with renaming in some super-function and separately for creation + separate function for old-style-path building to avoid the code copy. 3) Something better? [skip]