ManageIQ / pg-pglogical

A ruby gem for configuring and using pglogical
Apache License 2.0
3 stars 11 forks source link

Account for symbol table names in drop_table patch #19

Closed carbonin closed 6 years ago

carbonin commented 6 years ago

The array of tables names coming from the database contains only strings.

When comparing the given name to the list we were getting a false negative result and not removing the table from the replication set before trying to drop it.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1540528

miq-bot commented 6 years ago

Checked commit https://github.com/carbonin/pg-pglogical/commit/61f8a11174c2bc1c8b0bd64e987d107231f603eb with ruby 2.3.3, rubocop 0.52.0, haml-lint 0.20.0, and yamllint 1.10.0 1 file checked, 0 offenses detected Everything looks fine. :cake: