Skip to content

Triggerer Process Do Not create new trigger instance to run #21005

Closed

Description

@Greetlist

Apache Airflow version

2.2.1

What happened

When a Deferrable Task which is cleared or generated. This Trigger's TriggererId is NULL,and never change, which cause this trigger is never init and run by triggerer process.

What you expected to happen

When a Deferrable Task is generated, It will init and run correctly.

How to reproduce

Clear a Deferrable Task.
I add some log to source code and find out the reason of this problem:
image

First Query:
is None has a strange behavior, It does not work.
Second Query:
when the trigerer_id has a NULL value in database, it can not query correctly.

I fix both problems by:
image

Operating System

Ubuntu 16.04

Versions of Apache Airflow Providers

No response

Deployment

Other

Deployment details

1.One Triggerer Process
2. Three Scheduler
3. Pecona Mysql 8.0

Anything else

No response

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

Activity

  1. potiuk commented on Jan 23, 2022

    @potiuk
    Member

    Assigned you to it @Greetlist - can you please make a PR for that one with "Fixes: #21005" in the message (and cc: @andrewgodwin best to review it ), I thin you wil have to add a test case for that one but we can guide you.

  2. added this to the Airflow 2.3.0 milestone on Jan 23, 2022
  3. Greetlist commented on Jan 24, 2022

    @Greetlist
    Author

    I'm willing to make a PR for it, but I don't know how to add a test case. By the way, there is a test_trigger.py file in tests, Can I just use the file to test this case?

  4. potiuk commented on Jan 24, 2022

    @potiuk
    Member

    I believe so - but do not know too much about it - I think you can start with PR even without tests and the right people will comment there on how to add tests!

  5. Greetlist commented on Jan 25, 2022

    @Greetlist
    Author

    I have another PR #19640 which has not been Merged yet.I should wait until it is finished so I can start a new PR for this.

  6. removed this from the Airflow 2.3.3 milestone on Jul 3, 2022
  7. 6 remaining items

  8. added this to the Airflow 2.4.1 milestone on Sep 8, 2022
  9. modified the milestones: Airflow 2.4.3, on Nov 9, 2022
  10. modified the milestones: , Airflow 2.5.0, Airflow 2.5.1 on Nov 23, 2022
  11. removed this from the Airflow 2.5.1 milestone on Nov 25, 2022
  12. eladkal commented on Nov 25, 2022

    @eladkal
    Contributor

    @Greetlist are you working on this issue?

  13. Greetlist commented on Nov 26, 2022

    @Greetlist
    Author

    @eladkal I found this problem is solved by #21770. I close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions