daichirata/hammer

Suggestion: option to ignore `ALTER DATABASE` statement

gecko655 opened this issue · 0 comments

gecko655 commented

It is great for me if hammer has an option to ignore ALTER DATABASE statement while diff, apply, etc.

Backgrounds:

  • The go spanner library (currently hammer is using it) has 2 critical issues involving the ALTER DATABASE statement. These issues prevent me to use hammer completely.
  • ALTER DATABASE statement needs "database name" to run. it means an sql file that is synced to a database named "db1" cannot be directly used in a database named "db2", because the file has the DB name in SQL statement, like: 
    ALTER DATABASE db1 SET OPTIONS (version_retention_period='7d');
    It will be great for me to ignore the "ALTER DATABASE" statement to share/apply the same sql file.