Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge into the fork #5

Merged
merged 22 commits into from
Apr 16, 2019
Merged

Merge into the fork #5

merged 22 commits into from
Apr 16, 2019

Conversation

GoodDok
Copy link
Owner

@GoodDok GoodDok commented Apr 16, 2019

Description

Motivation and Context

Have you tested this? If so, how?

MichaelGrupp and others added 22 commits March 13, 2019 08:13
* Fix durations in D3 graph (fixes #2620)

Fixes the duration calculation. Both date objects `startTime` and
`finishTime` are in milliseconds, so the additional multiplication with
1000 was wrong.

Also, the duration is now calculated with:
   duration = last_updated - time_running
as it is done in datadog_metric.py:83

I decided to not display the time if the duration is longer than a day,
because then it gets messy with the Date objects (would require a
conversion to day of year, not day of month).

* Remove code duplication.
- changed `track_url_in_stderr` to `stream_for_searching_tracking_url` according to the latest changes in external_program.py
- added tests for spark url tracking in cluster and client modes
This fixes a DeprecationWarning which would have broken in Python 3.8.
…dules entry (#2692)

The configration doc contains entry on `[core]max_reschedules`,
which is not only deprecated (in favor of `[worker]max_reschedules`)
but also wrong: the correct (but deprecated) config key is
`[core]worker-max-reschedules`.
Better get rid of this all together and just keep `[worker]max_reschedules`
…task

add beam_dataflow_task to luigi/contrib
@GoodDok GoodDok merged commit 4129069 into GoodDok:master Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants