Merge pull request #1440 from ydb-platform/hide-eof-in-log

Hide EOF error in recvMsg logs
ydb-platform · Sep 4, 2024 · 0f5a484 · 0f5a484
2 parents 279aae7 + a94bb79
commit 0f5a484
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,5 @@
+* EOF error in RecvMsg is no longer logged
+
 ## v3.78.0
 * Changed result type of method `query.Executor.QueryResultSet` from `query.ResultSet` to `query.ClosableResultSet`
 * Added `table/types.DecimalValueFromString` decimal type constructor

diff --git a/log/driver.go b/log/driver.go
@@ -5,6 +5,7 @@ import (
 	"time"
 
 	"github.com/ydb-platform/ydb-go-sdk/v3/internal/secret"
+	"github.com/ydb-platform/ydb-go-sdk/v3/internal/xerrors"
 	"github.com/ydb-platform/ydb-go-sdk/v3/trace"
 )
 
@@ -302,7 +303,7 @@ func internalDriver(l Logger, d trace.Detailer) trace.Driver {
 			start := time.Now()
 
 			return func(info trace.DriverConnStreamRecvMsgDoneInfo) {
-				if info.Error == nil {
+				if xerrors.HideEOF(info.Error) == nil {
 					l.Log(ctx, "done",
 						latencyField(start),
 					)