Re: [PR] Add post endpoint for dataset events [airflow]

uranusjr merged PR #37570: URL: https://github.com/apache/airflow/pull/37570 -- 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:

Re: [PR] Add post endpoint for dataset events [airflow]

uranusjr commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1502109846 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

uranusjr commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1502109846 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1501958196 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

bbovenzi commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1500877439 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

edumuellerFSL commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1500430412 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -301,3 +312,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1500277317 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1500277317 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1500255255 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -311,3 +323,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1500254709 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -311,3 +323,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

dstandish commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1499630671 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

ephraimbuddy commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1499201237 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -311,3 +323,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1499158375 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1499158375 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1499149920 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -301,3 +312,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1499134237 ## airflow/datasets/manager.py: ## @@ -72,16 +72,20 @@ def register_dataset_change( if not dataset_model: self.log.warning("DatasetModel %s not

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1499116204 ## tests/api_connexion/schemas/test_dataset_schema.py: ## @@ -162,6 +162,37 @@ def test_serialize(self, session): } +class

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1498585177 ## tests/api_connexion/schemas/test_dataset_schema.py: ## @@ -162,6 +162,37 @@ def test_serialize(self, session): } +class

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1498585057 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -301,3 +312,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

Lee-W commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1497229772 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -301,3 +312,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

pankajkoti commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1496979912 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -301,3 +312,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

uranusjr commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1496977576 ## tests/api_connexion/schemas/test_dataset_schema.py: ## @@ -162,6 +162,37 @@ def test_serialize(self, session): } +class

Re: [PR] Add post endpoint for dataset events [airflow]

uranusjr commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1496976628 ## airflow/datasets/manager.py: ## @@ -56,8 +56,8 @@ def create_datasets(self, dataset_models: list[DatasetModel], session: Session)

Re: [PR] Add post endpoint for dataset events [airflow]

uranusjr commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1496975306 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -301,3 +312,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

uranusjr commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1496972515 ## airflow/api_connexion/endpoints/dataset_endpoint.py: ## @@ -301,3 +312,37 @@ def delete_dataset_queued_events( "Queue event not found",

Re: [PR] Add post endpoint for dataset events [airflow]

bbovenzi commented on code in PR #37570: URL: https://github.com/apache/airflow/pull/37570#discussion_r1496548285 ## airflow/www/static/js/components/Table/Cells.tsx: ## @@ -150,6 +150,9 @@ export const TaskInstanceLink = ({ cell: { value, row } }: CellProps) => { const {

Re: [PR] Add post endpoint for dataset events [airflow]

cmarteepants commented on PR #37570: URL: https://github.com/apache/airflow/pull/37570#issuecomment-1954979860 @Lee-W @pankajkoti -- 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

[PR] Add post endpoint for dataset events [airflow]

edumuellerFSL opened a new pull request, #37570: URL: https://github.com/apache/airflow/pull/37570 --- **^ Add meaningful description above** Read the **[Pull Request