Skip to content
This repository has been archived by the owner on Nov 26, 2021. It is now read-only.

Design thoughts on implementing spatial indexes #11

Open
Morgon opened this issue May 10, 2014 · 3 comments
Open

Design thoughts on implementing spatial indexes #11

Morgon opened this issue May 10, 2014 · 3 comments

Comments

@Morgon
Copy link

Morgon commented May 10, 2014

Using spatial functions would be a pretty huge win for MySQL users in this library.

Of course, there are a number of ways one could go about this, and since it's not my project, I thought I'd start some discussion on the best way to approach it.

When I took a crack at implementing a GeoNames database before finding this project, I simply added an additional POINT column in the main names table, and adding the SPATIAL INDEX. This of course requires the table be MyISAM [and be using MySQL], but I don't think that's a big deal - I doubt anyone will need transactional support on these tables, and I finally figured out how to get the current DB Driver in Laravel :)

In any case, could I follow that same convention to add it to the main table, or would you prefer it be external - either in a separately managed table, or completely out of scope of this module? It'll be trickier to do it this way, but I suppose it could ensure that the main tables were consistent, regardless of driver or settings.

@Morgon
Copy link
Author

Morgon commented Jun 10, 2014

Any thoughts here?

@ipalaus
Copy link
Owner

ipalaus commented Jun 10, 2014

I'll review it tomorrow morning, sorry for the delay @Morgon! Thanks in advance

@ipalaus
Copy link
Owner

ipalaus commented Jul 31, 2014

Sorry about this @Morgon, can you ping me on IRC anytime today or tomorrow?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants