-
Notifications
You must be signed in to change notification settings - Fork 624
Delete Overlay Flag #3211
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Delete Overlay Flag #3211
Conversation
return getDocumentsMatchingCollectionQueryFromMutationQueue(query, offset); | ||
} | ||
ImmutableSortedMap<DocumentKey, Document> fromOverlay = | ||
getDocumentsMatchingCollectionQueryFromOverlayCache(query, offset); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we inline this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
*/ | ||
return fromOverlay; | ||
} else { | ||
return getDocumentsMatchingCollectionQueryFromMutationQueue(query, offset); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This method can be removed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
Coverage ReportAffected SDKs
Test Logs
NotesHTML coverage reports can be produced locally with Head commit (469e3a56) is created by Prow via merging commits: e87b4c7 cb297fb. |
Binary Size ReportAffected SDKs
Test Logs
NotesHead commit (469e3a56) is created by Prow via merging commits: e87b4c7 cb297fb. |
/test smoke-tests |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for cleaning this up!
No description provided.