Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inclusive Naming: fix aya example and orc_test.sh #8941

Merged
merged 2 commits into from
Oct 9, 2021

Conversation

deepthi
Copy link
Member

@deepthi deepthi commented Oct 6, 2021

Description

Fix a few references in examples.

Related Issue(s)

#8618

Checklist

  • Should this PR be backported? NO
  • Tests were added or are not required
  • Documentation was added or is not required

Deployment Notes

Signed-off-by: deepthi <deepthi@planetscale.com>
@deepthi deepthi requested review from askdba and removed request for rohit-nayak-ps October 6, 2021 22:15
@@ -44,9 +44,6 @@ for i in 110 111; do
CELL=zone2 KEYSPACE=commerce TABLET_UID=$i ./scripts/ovttablet-up.sh
done

# set one of the replicas to primary
#vtctlclient InitShardMaster -force commerce/0 zone1-100
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this be replaced with a (commented-out 😂) InitShardPrimary call?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't think it was necessary since vtorc is supposed to automatically elect a primary.

Signed-off-by: deepthi <deepthi@planetscale.com>
@deepthi deepthi merged commit ad0d816 into vitessio:release-12.0 Oct 9, 2021
@deepthi deepthi deleted the ds-aya-naming branch October 9, 2021 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants