feat: add explain MongoDB queries #314
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
New Pull Request Checklist
Issue Description
The Swift SDK can't explain MongoDB queries due to a bug on the server-side response. PR parse-community/parse-server#7440 was created on the server, but the server team has chosen not to fix the code.
Originally discussed in: https://community.parseplatform.org/t/explain-result-in-parseswift/1557
Related issue: #N/A parse-community/parse-server#7442
Approach
Add a new internal response type that doesn't expect
results
to be an array. When the user wants to explain a mongo query, they set the flag,isUsingMongoDB=true
. This allows the mongo explained queries to use the same methods as the ones already in the SDK.TODOs before merging