Skip to content

Commit

Permalink
[AIRFLOW-682] Bump MAX_PERIODS to make mark_success work for large DAGs
Browse files Browse the repository at this point in the history
It is not possible to mark success on some large
DAGs due to the MAX_PERIODS being set to 1000. We
should temporarily bump it up until work can be
done to scale the mark success endpoint much
higher.

Testing Done:
- Has been running in Airbnb prod for many months
now

Closes apache#1928 from aoen/ddavydov/bump_max_periods
  • Loading branch information
aoen committed Dec 7, 2016
1 parent 64a1b19 commit 37a324a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion airflow/www/views.py
Original file line number Diff line number Diff line change
Expand Up @@ -1095,7 +1095,7 @@ def success(self):
future = request.args.get('future') == "true"
past = request.args.get('past') == "true"
recursive = request.args.get('recursive') == "true"
MAX_PERIODS = 1000
MAX_PERIODS = 5000

# Flagging tasks as successful
session = settings.Session()
Expand Down

0 comments on commit 37a324a

Please sign in to comment.