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

PS-9121 Innodb fails to update spatial index #5276

Closed
wants to merge 1 commit into from

Conversation

VarunNagaraju
Copy link
Contributor

https://perconadev.atlassian.net/browse/PS-9121

When a spatial index is used on a geomtery column which contains values with more precision, the comparison of minimum bounding rectangle(MBR) for the geometric values can result in unexpected errors because of trunation/rounding of data.

Since the underlying data type used for representing the coordinates of MBR is double, it supports upto 15 digits after the decimal place and any data after after those digits is subject to rounding and truncated.

So, the MBR comparison method borrowed from boost libaries is updated to hanlde comparisons upto 20 digits after the decimal place since the secondary index update relies on this comparison.

https://perconadev.atlassian.net/browse/PS-9121

When a spatial index is used on a geomtery column which
contains values with more precision, the comparison
of minimum bounding rectangle(MBR) for the geometric values
can result in unexpected errors because of trunation/rounding
of data.

Since the underlying data type used for representing the
coordinates of MBR is double, it supports upto 15 digits after
the decimal place and any data after after those digits is subject
to rounding and truncated.

So, the MBR comparison method borrowed from boost libaries is
updated to hanlde comparisons upto 20 digits after the decimal
place since the secondary index update relies on this comparison.
@VarunNagaraju
Copy link
Contributor Author

Name of the branch was wrong.

@VarunNagaraju VarunNagaraju deleted the PS-9141 branch April 2, 2024 14:38
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.

1 participant