tracyhenry/Kyrix

decouple indexing from createMV in indexers

peterg17 opened this issue · 3 comments

Essentially, because 3D indexing adds to the same table between canvases, I had trouble where the createMV function would overwrite the same index for each canvas. I solved this by building a workaround where I made a separate function for just the 3D rtree indexer, but I think that we should decouple the indexing from the other operations in createMV so this is more clean and well done. I understand this issue and can do it.

asah commented

sorry, I can't quite understand what the issue is.

createMV() is called per layer. As we talked, you can still follow this by only create table/index for layers that are not on the top-level, similar to what I did for my autoDDIndexer

Is this what this issue is about?

@peterg17 per discussion yesterday, we will find ways to infer whether a canvas is on the top/bottom level, to help decide whether to create index for this canvas. This will keep the current structure (i.e. indexing is still done in createMV).