Projections with more than 16 fields not implemented

Description

Hopefully I'm wrong and I'm just terrible at reading C# code, but I didn't see anything in Couchbase.CouchbaseCollection.GetAsync that falls back to full-document fetches and manually handle projections when there are more than 16 fields.

Link to discussion of falling back to full doc:
https://couchbase.slack.com/archives/CCX40TL9F/p1544705305558800

Environment

None

Gerrit Reviews

None

Release Notes Description

None

Activity

Show:

Jeffry Morris September 9, 2019 at 9:16 PM

No, it doesn't do that currently. It seems odd to go through and create a whole new doc based solely on the requested field and throw away the rest. Its going to fun explaining why we return all fields over the wire and then ditching the unrequested fields at the client.

Jeffry Morris September 9, 2019 at 8:03 PM

Line 262 is for projections if less than or equal to 16; on line 289 we use full doc if that condition is not met: https://github.com/couchbase/couchbase-net-client/blob/master/src/Couchbase/CouchbaseCollection.cs#L260-L305

Seems to be correct to me.

Fixed
Pinned fields
Click on the next to a field label to start pinning.

Details

Assignee

Reporter

Story Points

Sprint

Fix versions

Priority

Instabug

Open Instabug

PagerDuty

Sentry

Zendesk Support

Created August 28, 2019 at 10:48 PM
Updated April 24, 2020 at 8:23 PM
Resolved October 3, 2019 at 3:41 AM
Instabug