Skip to content
This repository has been archived by the owner on Feb 18, 2021. It is now read-only.

Bug fix: update is_multi_zone field in metadata for update destination request #222

Merged
merged 2 commits into from
Jun 14, 2017

Conversation

datoug
Copy link
Contributor

@datoug datoug commented Jun 12, 2017

We have two is_multi_zone field in destinations table, one in the destination type, another in a column in destinations. The bug is we only update the first field, which causes ListDestinations call to not return the mz destination after it's upgraded, because ListDestinations filters on the second filed.

CREATE TYPE destination (
uuid uuid,
path text,
type int,
status int,
consumed_messages_retention int,
unconsumed_messages_retention int,
owner_email text,
checksum_option int,
is_multi_zone boolean,
zone_configs list<frozen<destination_zone_config>>,
schema_version int
);

CREATE TABLE destinations (
uuid uuid PRIMARY KEY,
is_multi_zone boolean,
destination frozen,
)

@datoug datoug requested a review from kirg June 12, 2017 23:00
@datoug datoug merged commit 7b84ef5 into master Jun 14, 2017
@datoug datoug deleted the mz_fix branch June 14, 2017 16:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants