-
Notifications
You must be signed in to change notification settings - Fork 19
Open
Labels
api: bigquerystorageIssues related to the googleapis/nodejs-bigquery-storage API.Issues related to the googleapis/nodejs-bigquery-storage API.
Description
Please make sure you have searched for information in the following guides.
- Search the issues already opened: https://github.com/GoogleCloudPlatform/google-cloud-node/issues
- Check our Troubleshooting guide: https://googlecloudplatform.github.io/google-cloud-node/#/docs/guides/troubleshooting
- Check our FAQ: https://googlecloudplatform.github.io/google-cloud-node/#/docs/guides/faq
- Check our libraries HOW-TO: https://github.com/googleapis/gax-nodejs/blob/main/client-libraries.md
- Check out our authentication guide: https://github.com/googleapis/google-auth-library-nodejs
- Check out handwritten samples for many of our APIs: https://github.com/GoogleCloudPlatform/nodejs-docs-samples
Documentation Request
I think it is better to delete streamId in JSONWriter constructor in README.md
README.md is writed about blew.
const streamId = connection.getStreamId();
const writer = new JSONWriter({
streamId,
connection,
protoDescriptor,
});But WriterOptions has not streamId of properties and JSONWriter not use streamId.
https://github.com/googleapis/nodejs-bigquery-storage/blob/v5.1.0/src/managedwriter/writer.ts#L36
So, if we use typescript, then it occures error of typescript.
I think it is better to delete streamId of JSONWriter in README.md if we don't use it in the future
Thank you for your reading this issue 🙇
If there is a common understanding and no one is available, I will create a PR for README.md and related documents.
Metadata
Metadata
Assignees
Labels
api: bigquerystorageIssues related to the googleapis/nodejs-bigquery-storage API.Issues related to the googleapis/nodejs-bigquery-storage API.