Skip to content

Conversation

@tywalch
Copy link
Owner

@tywalch tywalch commented Mar 28, 2024

This branch fixes issue #362. The root cause of the issue was that one of the composite attributes was hidden. Because the cursor was created using the "formatted" return item, it was the hidden composite was not available to the cursor formatting function. We now check for this unique case and perform some additional calculations. Ideally this reduces the surface area of performance implications to only entities that have a hidden composite attribute defined.

This branch fixes issue #362. The root cause of the issue was that one of the composite attributes was hidden. Because the cursor was created using the "formatted" return item, it was the hidden composite was not available to the cursor formatting function. We now check for this unique case and perform some additional calculations. Ideally this reduces the surface area of performance implications to only entities that have a hidden composite attribute defined.
@tywalch
Copy link
Owner Author

tywalch commented Mar 28, 2024

TODO: unit tests

@netlify
Copy link

netlify bot commented Mar 28, 2024

Deploy Preview for electrodb-dev canceled.

Name Link
🔨 Latest commit a6d6c9e
🔍 Latest deploy log https://app.netlify.com/sites/electrodb-dev/deploys/6605ce8ad4a3620008e77328

@tywalch tywalch linked an issue Mar 28, 2024 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Malformed cursor causing issues with paginated queries

2 participants