Details
-
Bug
-
Status: Closed
-
Major
-
Resolution: Fixed
-
Cheshire-Cat, 6.6.0
-
Untriaged
-
1
-
Unknown
Description
N1QL/FTS use grpc version 1.17.
There seems to be a leak within grpc's resetTransport as pointed out here -
- https://github.com/grpc/grpc-go/issues/3667
- https://github.com/etcd-io/etcd/issues/11371
- https://github.com/grpc/grpc-go/issues/2976
which is addressed by https://github.com/grpc/grpc-go/pull/2985 for version 1.24
Consider upgrading grpc within these projects to v1.24.0?
- https://github.com/couchbase/cbft
- https://github.com/couchbase/cbftx
- https://github.com/couchbase/query
- https://github.com/couchbase/n1fty
The leak is coming from context.WithCancel as pointed out here: https://github.com/grpc/grpc-go/issues/2976.
Setting a timeout with context is a workaround for this - by setting a deadline for the datastore.IndexConnection which is handed to n1fty's Search(..). But WithTimeout for context was introduced in 7.0, so this is not a viable workaround for older builds.
Attachments
Activity
Link | This issue relates to CBSE-9208 [ CBSE-9208 ] |
Fix Version/s | 6.6.2 [ 17103 ] |
Summary | Connection leak within gRPC's resetTransport | Memory leak within gRPC's resetTransport |
Summary | Memory leak within gRPC's resetTransport | Memory leak within gRPC's resetTransport(..) |
Summary | Memory leak within gRPC's resetTransport(..) | Memory leak within gRPC's resetTransport(..) due to context without Timeout |
Summary | Memory leak within gRPC's resetTransport(..) due to context without Timeout | Memory leak within gRPC's resetTransport(..) |
Description |
N1QL/FTS use grpc version 1.17.
There seems to be a leak within grpc's resetTransport as pointed out here - * [https://github.com/grpc/grpc-go/issues/3667] * [https://github.com/etcd-io/etcd/issues/11371] * [https://github.com/grpc/grpc-go/issues/2976] which is addressed by [https://github.com/grpc/grpc-go/pull/2985] for version 1.24 Consider upgrading grpc within these projects to v1.24.0? * [https://github.com/couchbase/cbft|http://github.com/couchbase/cbft] * [https://github.com/couchbase/cbftx] * [https://github.com/couchbase/query|http://github.com/couchbase/query] * [https://github.com/couchbase/n1fty|http://github.com/couchbase/n1fty] The leak is coming from context.WithCancel as pointed out here: [https://github.com/grpc/grpc-go/issues/2976]. Setting a timeout with context is a workaround for this - by setting a deadline for the datastore.IndexConnection which is handed to n1fty's Search(..). |
N1QL/FTS use grpc version 1.17.
There seems to be a leak within grpc's resetTransport as pointed out here - * [https://github.com/grpc/grpc-go/issues/3667] * [https://github.com/etcd-io/etcd/issues/11371] * [https://github.com/grpc/grpc-go/issues/2976] which is addressed by [https://github.com/grpc/grpc-go/pull/2985] for version 1.24 Consider upgrading grpc within these projects to v1.24.0? * [https://github.com/couchbase/cbft|http://github.com/couchbase/cbft] * [https://github.com/couchbase/cbftx] * [https://github.com/couchbase/query|http://github.com/couchbase/query] * [https://github.com/couchbase/n1fty|http://github.com/couchbase/n1fty] The leak is coming from context.WithCancel as pointed out here: [https://github.com/grpc/grpc-go/issues/2976]. Setting a timeout with context is a workaround for this - by setting a deadline for the datastore.IndexConnection which is handed to n1fty's Search(..). But WithTimeout for context was introduced in 7.0, so this is not a viable workaround for older builds. |
Summary | Memory leak within gRPC's resetTransport(..) | Memory leak within gRPC's resetTransport(..) due to context.WithCancel(..) |
Fix Version/s | 6.6.1 [ 17002 ] | |
Fix Version/s | 6.6.2 [ 17103 ] |
Labels | approved-for-6.6.1 |
Link | This issue blocks MB-40528 [ MB-40528 ] |
Resolution | Fixed [ 1 ] | |
Status | Open [ 1 ] | Resolved [ 5 ] |
Assignee | Abhinav Dangeti [ abhinav ] | Mihir Kamdar [ mihir.kamdar ] |
Status | Resolved [ 5 ] | Closed [ 6 ] |
Fix Version/s | 7.0.0 [ 17233 ] |
Fix Version/s | Cheshire-Cat [ 15915 ] |
For 7.0 builds ..