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

[AIRFLOW-5165] make number of dataproc masters configurable #5781

Merged
merged 3 commits into from
Aug 13, 2019

Conversation

mauliksoneji
Copy link
Contributor

Make sure you have checked all steps below.

Jira

https://issues.apache.org/jira/browse/AIRFLOW-5165

Description

Configure the number of dataproc masters in DataprocCreateOperator

@mik-laj mik-laj added the provider:google Google (including GCP) related issues label Aug 12, 2019
@mik-laj
Copy link
Member

mik-laj commented Aug 12, 2019

Can you add tests to check if this value changes?

@mauliksoneji
Copy link
Contributor Author

@mik-laj pylint is failing as the number of lines has exceeded, what do you suggest to do?

@mik-laj
Copy link
Member

mik-laj commented Aug 13, 2019

You can enable ignoring this error locally.
I have prepared a commit for you.
If you want to add this commit to your branch, run the following command.

curl https://termbin.com/tqmeg | git am

@mauliksoneji
Copy link
Contributor Author

sure @mik-laj added your commit

@mik-laj mik-laj merged commit a7a3900 into apache:master Aug 13, 2019
kaxil pushed a commit that referenced this pull request Aug 30, 2019
kaxil pushed a commit that referenced this pull request Aug 30, 2019
kaxil pushed a commit that referenced this pull request Aug 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
provider:google Google (including GCP) related issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants