-
Notifications
You must be signed in to change notification settings - Fork 14.8k
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-599] Adding spotify to Airflow Users #1855
Conversation
Current coverage is 66.36% (diff: 100%)@@ master #1855 diff @@
==========================================
Files 127 127
Lines 9772 9772
Methods 0 0
Messages 0 0
Branches 0 0
==========================================
Hits 6485 6485
Misses 3287 3287
Partials 0 0
|
Hi @znichols Please add Spotify in alphabetic order to the file. Also, refer to https://cwiki.apache.org/confluence/display/AIRFLOW/Contributors%27+Guide Even for trivial commits such as this one, we'd like the community to create a JIRA and reference it in the commit message (e.g. [AIRFLOW-XXX]) as a prefix. |
@r39132 Updated the PR and added relevant ticket here: Apologies for not taking a closer look at the (pretty nice) guidelines. |
+1 @znichols Welcome to the community! |
Dear Airflow Maintainers, Please accept this PR that addresses the following issues: - Adds Spotify to list of Airflow users Closes apache#1855 from znichols/spotify_use
Dear Airflow Maintainers, Please accept this PR that addresses the following issues: - Adds Spotify to list of Airflow users Closes apache#1855 from znichols/spotify_use
Spotify uses his own pipeline tool. How can be posible that they also uses Airflow? Any reference on this? |
@marctc They actually use multiple(like: https://github.com/spotify/styx). We do as well (more in the proprietary space). Why don't you think that is possible? |
Dunno. I thought that they standardized his tools and the whole company uses the same one. |
Dear Airflow Maintainers,
Please accept this PR that addresses the following issues: