Mushegh Malkhasyan

Historically, it was a single clock definition - GETTIME which could be mapped to anything (or overridden like in case of testing or other runtimes). Later, a new time function...

I agree it should. Ideally, there should be a new entry in common defs called GETMONOTONICTIME which defaults to the monotonic clock and this is what should be used instead...

> > Looks OK. Is there any scenario that we might be breaking with this? > > Like a scene I just met, the turn server is deployed in the...

I provided some feedback on https://github.com/awslabs/amazon-kinesis-video-streams-producer-c/pull/254#issuecomment-956453242 Also, the PIC solution to jitter has a threshold below which jitter is not applied. Is there a good reason to add jitter to...

@odellcraig - great suggestion! Do you mind sending a pull request for this?

@dswamina do you have any further update on this issue? @deepaksaxenakent has your original issue been resolved? I would like to get more traction on this issue and not let...

Can this item be resolved please? I think Jeffrey has a comprehensive response and we would like to not proliferate features that do not have clear and consistent benefits across...

Understood. My stance on this is the "support" for a particular feature is somewhat different than having a prescriptive guidance in the API. The API guidance should cover the cases...

I see the PR Description Check failed but I am at a loss as to what exactly needs to be provided there and why this was flagged as "short". Help?

> Nice! > > I looked into the TSAN that's failing it's happening across two different test runs. > > The testThreadRoutine thread from the test `ThreadCreateAndCancel` is still running...