Skip to content

Conversation

@saurabhdhupar
Copy link
Contributor

@saurabhdhupar saurabhdhupar commented Feb 25, 2020


Issue link: AIRFLOW-5485

Make sure to mark the boxes below before creating PR: [x]

  • [x ] Description above provides context of the change
  • [ x] Commit message/PR title starts with [AIRFLOW-NNNN]. AIRFLOW-NNNN = JIRA ID*
  • [x ] Unit tests coverage for changes (not needed for documentation changes)
  • [ x] Commits follow "How to write a good git commit message"
  • [x ] Relevant documentation is updated including usage instructions.
  • [x ] I will engage committers as explained in Contribution Workflow Example.

* For document-only changes commit message can start with [AIRFLOW-XXXX].


In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.
Read the Pull Request Guidelines for more information.

@boring-cyborg boring-cyborg bot added the area:Scheduler including HA (high availability) scheduler label Feb 25, 2020
@saurabhdhupar saurabhdhupar changed the title Fix for Airflow 5485 Airflow 5485 Feb 25, 2020
@saurabhdhupar saurabhdhupar changed the title Airflow 5485 AIRFLOW-5485 Feb 25, 2020
@saurabhdhupar saurabhdhupar changed the title AIRFLOW-5485 [AIRFLOW-5485] - scheduler_job: replace == None with is Feb 25, 2020
heartrate = conf.getint('scheduler', 'SCHEDULER_HEARTBEAT_SEC')

def __init__(
def __init__(
Copy link
Contributor

@Fokko Fokko Feb 25, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this additional space will cause a lint error. Please revert this line.

@Fokko
Copy link
Contributor

Fokko commented Feb 25, 2020

Thanks @saurabhdhupar for the contribution. Let's wait for the CI.

@mik-laj
Copy link
Member

mik-laj commented Feb 25, 2020

I did it earlier in this PR: #7527

@ashb
Copy link
Member

ashb commented Feb 25, 2020

The other PR has been merged, so I'm closing this. Thanks for the PR though!

@ashb ashb closed this Feb 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Scheduler including HA (high availability) scheduler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants