-
Notifications
You must be signed in to change notification settings - Fork 185
docs: Updated production.md
to reflect it is not possible to set this configuration value
#9468
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
base: main
Are you sure you want to change the base?
Conversation
It's is not possible to configure the local executor via the `meltano.yml` configuration.
✅ Deploy Preview for meltano ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideRemoved an unsupported configuration example from production.md by deleting the ‘meltano config airflow set core.executor LocalExecutor’ snippet, ensuring the docs only reference valid settings. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
production.md
to reflect it is not possible to set this configuration valueproduction.md
to reflect it is not possible to set this configuration value
Description
The current "go to production" docs explain how to configure the Airflow executor using a configuration option that is not available to set via according to the documentation here: https://hub.meltano.com/orchestrators/airflow/ (only the prior discussed configuration option
sql_alchemy_conn
can be set).I expect this PR won't be merged as is because I've kind of clobbered the documentation but I wanted to raise this issue as it caused me a few hours of trying to figuring out why it wasn't being accepted.
Summary by Sourcery
Documentation: