Skip to content

Add blog with Airflow Survey results#219

Merged
mik-laj merged 6 commits intoapache:aip-11from
turbaszek:blog-airflow-survey
Dec 18, 2019
Merged

Add blog with Airflow Survey results#219
mik-laj merged 6 commits intoapache:aip-11from
turbaszek:blog-airflow-survey

Conversation

@turbaszek
Copy link
Member

No description provided.

Copy link

@TobKed TobKed left a comment

Choose a reason for hiding this comment

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

Good job! I like it a lot.

@turbaszek
Copy link
Member Author

@mik-laj can you please take a look

Copy link
Contributor

@louisguitton louisguitton left a comment

Choose a reason for hiding this comment

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

really insightful thank you

Comment on lines 53 to 55
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm not sure I follow you here.
5 people use airflow for devops only ✅
this doesn't imply that all the other respondents use it for DevOps and something else . A respondent can use it for ETL and ML and not DevOps

Copy link
Member Author

@turbaszek turbaszek Dec 18, 2019

Choose a reason for hiding this comment

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

I think it implies that because 64 users selected "devops" but 5 of them selected only this option meaning that the rest, 55 users use Airflow for both devops and ETL / ML. I can agree that direction of this implication is only an assumption based on results.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah ok I understand what you meant: so "if you use it for DevOps, you don't use it only for DevOps". But there are still 308-64 people that don't care about DevOps so to speak

Copy link
Member

Choose a reason for hiding this comment

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

I 'd also remove the last sentence :) it's not obvious.

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 reformulated it

Copy link
Member

@potiuk potiuk left a comment

Choose a reason for hiding this comment

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

Looks good! just two NITS.

Comment on lines 53 to 55
Copy link
Member

Choose a reason for hiding this comment

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

I 'd also remove the last sentence :) it's not obvious.

@mik-laj mik-laj merged commit c8c76db into apache:aip-11 Dec 18, 2019
mik-laj pushed a commit that referenced this pull request Dec 18, 2019
potiuk pushed a commit that referenced this pull request May 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

Comments