You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "bbovenzi (via GitHub)" <gi...@apache.org> on 2023/08/01 15:14:22 UTC
[GitHub] [airflow] bbovenzi opened a new pull request, #33009: Merge datasets and dag dependencies graphs
bbovenzi opened a new pull request, #33009:
URL: https://github.com/apache/airflow/pull/33009
The Dag Dependencies and Datasets views had a lot of overlap codewise and functionality wise. We should just merge them.
So now the datasets page includes trigger and sensors and the old dag dependencies view is deleted.
I changed the url and the header name. But very much open to suggestions.
<img width="1458" alt="Screenshot 2023-08-01 at 11 06 40 PM" src="https://github.com/apache/airflow/assets/4600967/52adecf5-67cf-41c0-9691-f0c49eac769f">
---
**^ Add meaningful description above**
Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst#pull-request-guidelines)** for more information.
In case of fundamental code changes, an Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals)) is needed.
In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x).
In case of backwards incompatible changes please leave a note in a newsfragment file, named `{pr_number}.significant.rst` or `{issue_number}.significant.rst`, in [newsfragments](https://github.com/apache/airflow/tree/main/newsfragments).
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Merge datasets and dag dependencies graphs [airflow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] closed pull request #33009: Merge datasets and dag dependencies graphs
URL: https://github.com/apache/airflow/pull/33009
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] RNHTTR commented on pull request #33009: Merge datasets and dag dependencies graphs
Posted by "RNHTTR (via GitHub)" <gi...@apache.org>.
RNHTTR commented on PR #33009:
URL: https://github.com/apache/airflow/pull/33009#issuecomment-1661986838
> > Brent and I had a chat about this, and I'm just not sure on the naming when combining these 2 pretty distinct conceptual views,
>
> What about lineage view?
IMO "lineage" is maybe a bit overloaded, but maybe being a bit more specific, like "DAG Lineage" could be good.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] eladkal commented on pull request #33009: Merge datasets and dag dependencies graphs
Posted by "eladkal (via GitHub)" <gi...@apache.org>.
eladkal commented on PR #33009:
URL: https://github.com/apache/airflow/pull/33009#issuecomment-1661529553
> Brent and I had a chat about this, and I'm just not sure on the naming when combining these 2 pretty distinct conceptual views,
What about lineage view?
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Merge datasets and dag dependencies graphs [airflow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #33009:
URL: https://github.com/apache/airflow/pull/33009#issuecomment-1793894486
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] bbovenzi commented on pull request #33009: Merge datasets and dag dependencies graphs
Posted by "bbovenzi (via GitHub)" <gi...@apache.org>.
bbovenzi commented on PR #33009:
URL: https://github.com/apache/airflow/pull/33009#issuecomment-1726346729
@jedcunningham let's chat about this again for 2.8?
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] github-actions[bot] commented on pull request #33009: Merge datasets and dag dependencies graphs
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #33009:
URL: https://github.com/apache/airflow/pull/33009#issuecomment-1722347853
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [airflow] jedcunningham commented on pull request #33009: Merge datasets and dag dependencies graphs
Posted by "jedcunningham (via GitHub)" <gi...@apache.org>.
jedcunningham commented on PR #33009:
URL: https://github.com/apache/airflow/pull/33009#issuecomment-1661372267
Brent and I had a chat about this, and I'm just not sure on the naming when combining these 2 pretty distinct conceptual views, even if they share a common component. I also think with the timing around 2.7, we'd be better waiting a bit to make sure we get it right.
--
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-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org