-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Refactor BulkWriter retries #15385
Copy link
Copy link
Open
Labels
api: firestoreIssues related to the Firestore API.Issues related to the Firestore API.priority: p3Desirable enhancement or fix. May not be included in next release.Desirable enhancement or fix. May not be included in next release.type: cleanupAn internal cleanup or hygiene concern.An internal cleanup or hygiene concern.
Metadata
Metadata
Assignees
Labels
api: firestoreIssues related to the Firestore API.Issues related to the Firestore API.priority: p3Desirable enhancement or fix. May not be included in next release.Desirable enhancement or fix. May not be included in next release.type: cleanupAn internal cleanup or hygiene concern.An internal cleanup or hygiene concern.
Type
Fields
Give feedbackNo fields configured for issues without a type.
There are a number of issues with the current BulkWriter retry implementation:
I sugest a full internal refactor of the class, keeping the API the same but cleaning up the internals, and surfacing some extra user configuration