Skip to content

Adds Deadline.time_exceeded\1#5

Open
adkron wants to merge 1 commit into
elixir-toniq:masterfrom
BinaryNoggin:adds_time_exceeded
Open

Adds Deadline.time_exceeded\1#5
adkron wants to merge 1 commit into
elixir-toniq:masterfrom
BinaryNoggin:adds_time_exceeded

Conversation

@adkron
Copy link
Copy Markdown

@adkron adkron commented Sep 2, 2021

I often want to get the time that we have blown past
the deadline for logging, debugging, or gathering more
information about the response times of downstream services. Since we
are using Deadline for ensuring that we don't continue to wait around
I often find myself pulling the deadline time out of the context and
then copying how Deadline.time_remaining\1 works, but swapping the
subtraction. It would be nice not to worry about Deadline's internal workings
and get the information straight from Deadline.

#NogginDay

I often find myself wanting to get the time that we have blown past
the deadline for logging, debugging, or generally gathering more
information about the response times of down stream services. Since we
are using Deadline for ensuring that we don't continue to wait around
I often find myself pulling the deadline time out of the context and
then copying how `Deadline.time_remaining\1` works, b ut swapping the
subtraction. It would nice to be able to not have to worry about the
internal workings of Deadline and get the information straight from Deadline.
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.

1 participant