Skip to content

Fixes 362#368

Open
tywalch wants to merge 1 commit into
masterfrom
issue/362
Open

Fixes 362#368
tywalch wants to merge 1 commit into
masterfrom
issue/362

Conversation

@tywalch

@tywalch tywalch commented Mar 28, 2024

Copy link
Copy Markdown
Owner

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

tywalch commented Mar 28, 2024

Copy link
Copy Markdown
Owner Author

TODO: unit tests

@netlify

netlify Bot commented Mar 28, 2024

Copy link
Copy Markdown

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