[PR] Update _index.md [airflow-site]
navrang9 opened a new pull request, #1019: URL: https://github.com/apache/airflow-site/pull/1019 (no comment) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update _index.md [airflow-site]
dabla closed pull request #933: Update _index.md URL: https://github.com/apache/airflow-site/pull/933 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update _index.md [airflow-site]
dabla commented on PR #933: URL: https://github.com/apache/airflow-site/pull/933#issuecomment-2048037537 > I think we can close this PR in favor of [apache/airflow#38111](https://github.com/apache/airflow/pull/38111) ? Hey Elad yes this one can be closed. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update _index.md [airflow-site]
eladkal commented on PR #933: URL: https://github.com/apache/airflow-site/pull/933#issuecomment-2044284770 I think we can close this PR in favor of https://github.com/apache/airflow/pull/38111 ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update _index.md [airflow-site]
dabla commented on PR #933: URL: https://github.com/apache/airflow-site/pull/933#issuecomment-1890413832 > We have multiple DAG's interacting with MS Graph API endpoints, and as we want to avoid custom code as much as possible as we have to handle lot's of data due to paging. We though of implementing an operator for it using the official python client from [Microsoft](https://github.com/microsoftgraph/msgraph-sdk-python), that way we can simplify our dags and remove custom code as much as possible. Discussion started in d...@airflow.apache.org -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update _index.md [airflow-site]
potiuk commented on PR #933: URL: https://github.com/apache/airflow-site/pull/933#issuecomment-1890389560 Please start discussion about it at the devlist https://airflow.apache.org/community/ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update _index.md [airflow-site]
dabla commented on PR #933: URL: https://github.com/apache/airflow-site/pull/933#issuecomment-1890379876 > Well. one comment here @dabla - I think you should not publish it under `apache-airflow-providers-*` name. While we do not have a way to prevent it - this might be misleading to publish it with the `apache-airflow-` prefix, that it is an Apache Software Foundation project. > > Since you are managing it on your own - I would strongly suggest you to choose a different name and remove the `apache-airflow-*` or - even better - remove it and pass the ownership of the project to the Apache Airflow organization so that it is not misused. > > This might change in the future, where organization and provenance of the packages in PyPi is more visible, also we do not have written rules about it, so this is more of a kind request rather than requirement, but I think it would be good to follow this approach - that is currently pretty much universal - where `apache-airflow-` prefix is for the projects that are managed by the Apache Airflow PMC. > > Would that be possible @dabla ? Hello Jarek (@potiuk), Well if possible I would like to hand over the provider to the Apache Airflow project, as this was the whole point from the start, at least if there is interest in it... otherwise I will rename the provider and remove the apache prefixed one from pypi. I started a discussion about that operator at the end of last year at the Airflow project: https://github.com/apache/airflow/discussions/36315 While awaiting a decission about that discussion (a vote is required if I understood well), I opted to already deploy it to pypi so it is open source and others can profit from it as well and maybe even improve it and do contributions. If Airflow would take over the provider that would of course be even better. We have multiple DAG's interacting with MS Graph API endpoints, and as we want to avoid custom code as much as possible as we have to handle lot's of data due to paging. We though of implementing an operator for it using the official python client from [Microsoft](https://github.com/microsoftgraph/msgraph-sdk-python), that way we can simplify our dags and remove custom code as much as possible. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update _index.md [airflow-site]
potiuk commented on PR #933: URL: https://github.com/apache/airflow-site/pull/933#issuecomment-1889785426 Well. one comment here @dabla - I think you should not publish it under `apache-airlfow-providers`. While we do not have a way to prevent it - this might be misleading to publish it with the `apache-airflow-` prefix, that it is an Apache Software Foundation project. Since you are managing it on your own - I would strongly suggest you to choose a different name and remove the `apache-airflow-*` or - even better - remove it and pass the ownership of the project to the Apache Airflow organization so that it is not misused. This might change in the future, where organization and provenance of the packages in PyPi is more visible, also we do not have written rules about it, so this is more of a kind request rather than requirement, but I think it would be good to follow this approach - that is currently pretty much universal - where `apache-airflow-` prefix is for the projects that are managed by the Apache Airflow PMC. Would that be possible @dabla ? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[PR] Update _index.md [airflow-site]
dabla opened a new pull request, #933: URL: https://github.com/apache/airflow-site/pull/933 Added new entry in the Async Providers for the Airflow MS Graph API provider -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
Re: [PR] Update _index.md [airflow-site]
eladkal merged PR #869: URL: https://github.com/apache/airflow-site/pull/869 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
[PR] Update _index.md [airflow-site]
martinibach opened a new pull request, #869: URL: https://github.com/apache/airflow-site/pull/869 We would like to add Restack as an Airflow as a Service provider going forward. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org