FR: allow DocumentReference.select similar to Query.select about nodejs-firestore HOT 7 CLOSED

We are currently debating on how to best implement this. Please stay tuned.

from nodejs-firestore.

I have also filed this against firebase/firebase-js-sdk#272 since I am not sure where the source of development on the firestore part is housed.

from nodejs-firestore.

firebase/firebase-js-sdk#212 answers this

from nodejs-firestore.

@goldensunliu Are you sure that that the issue in the Firebase JS SDK really answers this?
As I understood the answer from @wilhuff, is that this won't appear in the JS CLIENT SDK as you are having offline capabilities there and so on.

I don't see a problem in the ADMIN SDK to add such a method to the DocumentReference. As far as I know, there is no caching in the ADMIN SDK, so it should be no problem to allow a snippet/method like this:

admin.firestore().document("path/to/a/document").select("y").get().then(snap => {
    console.log(`snapShot only has one field ${snap.data()}`);
    // Expected output: {y: 5}
});

Or am I wrong?

from nodejs-firestore.

@IchordeDionysos you're right: the comments in firebase/firebase-js-sdk#212 specifically relate to the mobile clients and the complications of interacting with the cache while offline. These concerns do not apply to the nodejs or admin SDKs.

document(..).get() translates into a BatchGetDocumentsRequest RPC, which includes a document mask, so this should be possible to implement.

In the meantime you can work around this by writing the following instead:

const projectId = // ... your project id 
const path = 'path/to/a/document';
db.document(path)
    .parent  // i.e. the collection containing the document
    .where('__name__', '==', `projects/${projectId}/databases/(default)/documents/${path}`)
    .select(...)

Note that the result will be a query snapshot rather than a document snapshot, but it should get you what you want in the short term.

@schmidt-sebastian holler if I got this wrong.

from nodejs-firestore.

A more concise version would be as follows:

const docRef = db.doc('path/to/a/document');
docRef.parent.where(Firestore.FieldPath.documentId(), '==', docRef).select(...)

from nodejs-firestore.

It would be great to have this issue re-opened for consideration.

As mentioned above, the comments in firebase/firebase-js-sdk#212 don’t apply to the Admin SDK. It would be great to have a DocumentReference.select() function implemented there.

from nodejs-firestore.