Skip to content

Commit a4ed606

Browse files
authored
fix: Minor fix to "fix: Evict inference request internal release callback after invoking (#452)"
1 parent 9f6c3ca commit a4ed606

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

src/infer_request.cc

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -476,10 +476,9 @@ InferenceRequest::Release(
476476
std::unique_ptr<InferenceRequest>&& request, const uint32_t release_flags)
477477
{
478478
// Invoke the release callbacks added internally before releasing the
479-
// request to user provided callback.
480-
481-
// Invoke callbacks in reverse order. Evict internal callbacks for reusing
482-
// inference request object.
479+
// request to user provided callback. Remove internal callbacks registered
480+
// through AddInternalReleaseCallback to allow reuse of the inference request
481+
// object.
483482
auto& release_callbacks = request->release_callbacks_;
484483
for (int i = release_callbacks.size() - 1; i >= 0; --i) {
485484
auto [release_fn, is_internal] = release_callbacks[i];
@@ -508,7 +507,6 @@ InferenceRequest::Release(
508507
"Failed to set released state");
509508
void* userp = request->release_userp_;
510509
auto& release_fn = request->release_fn_;
511-
LOG_INFO << "userp " << userp << std::endl;
512510
release_fn(
513511
reinterpret_cast<TRITONSERVER_InferenceRequest*>(request.release()),
514512
release_flags, userp);

0 commit comments

Comments
 (0)