| Commit message (Collapse) | Author | Age |
|
|
|
|
|
| |
They don't make sense in the open source repository.
PiperOrigin-RevId: 183140889
|
|
|
|
|
|
| |
isn't supported
PiperOrigin-RevId: 166364134
|
|
|
|
| |
PiperOrigin-RevId: 157628970
|
|
|
|
| |
PiperOrigin-RevId: 157591364
|
|
|
|
|
|
| |
"virtual")
PiperOrigin-RevId: 157284709
|
|
|
|
|
|
| |
cost_graph_ is stored.
Change: 154224628
|
|
|
|
| |
Change: 153637886
|
|
|
|
|
| |
usage remains bounded over time.
Change: 153123196
|
|
|
|
| |
Change: 152974181
|
|
|
|
|
| |
Make the queue runner own the metadata and mutex.
Change: 150475730
|
|
|
|
| |
Change: 149388619
|
|
|
|
| |
Change: 148954491
|
|
|
|
|
| |
so it runs faster (11s on tsan instead of ~30s).
Change: 147893428
|
|
|
|
|
| |
Add a new tensorflow::Status::IgnoreError() method to mark call sites where a Status has been intentionally ignored.
Change: 147402405
|
|
|
|
|
|
|
|
|
|
|
|
| |
This patch:
- Updates // comments to ///. I manually reverted some comments that shouldn't be docs (e.g. TODOs), but may have missed some.
- Indents code blocks so they get formatted as such in the docs.
- Removes /* */ comments from example code since it messes up Doxygen.
- Puts a space between {{ and }} since it messes up devsite.
- Adds some // START_SKIP_DOXYGEN and // END_SKIP_DOXYGEN comments for functions that aren't part of the public API (incomplete)
This will likely require further small fixups, but this gets something to be generated.
Change: 144749351
|
|
|
|
| |
Change: 141499300
|
|
|
|
| |
Change: 141484062
|
|
|
|
|
| |
ops since the cancel op will close the queue.
Change: 141122341
|
|
|
|
|
| |
case of an error
Change: 140478026
|
|
|
|
| |
Change: 140477117
|
|
|
|
| |
Change: 140380090
|
|
|
|
| |
Change: 138467240
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Three bug fixes:
(1) There was a thread-unsafe access to runs_ which could result in the
queue close operation being invoked multiple times.
(2) The Run() loop would not exit when there were multiple threads
and the queue was closed (i.e., the enqueue failed with a
queue_closed_exception_types_ error).
Without this fix, the changed QueueRunnerTest.QueueCloseCode
test would fail with a timeout since qr->Join() would be blocked
on the never-exiting Run() call
(3) Errors in invoking the close operation were being ignored.
Without this fix, the added QueueRunnerTest.QueueCloseFails
test would fail as Join() would return OK instead of NOT_FOUND
Two other minor changes:
- Slight simplification to QueueRunner::Run() so that
runs_ is manipulated only once and the body of the loop
is clearer
- Avoid starting an extra thread which will not be used
when there is no Coordinator. (Though in practice I
suppose we always intend to have a coordinator).
Change: 138228243
|
|
|
|
| |
Change: 137866409
|
|
|
|
|
| |
QueueRunner::Start(). This provides a reliable way to check the value of the status_ variable.
Change: 137769682
|
|
|
|
| |
Change: 137749204
|
|
|
|
| |
Change: 137731250
|
|
|
|
| |
Change: 137481049
|
|
|
|
| |
Change: 137447384
|
|
|
|
| |
Change: 137059071
|
|
Change: 136769119
|