Skip to content
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

Fixes 362 #368

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fixes 362 #368

wants to merge 1 commit into from

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

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
1 participant