Valery Mironov

No need this functions for now

@jmr Here I need to create external S2RegionCoverer: https://github.com/arangodb/arangodb/blob/d37f3d17f69dd388464cd509b749c771f01e1c55/arangod/IResearch/GeoAnalyzer.h#L73 Because for some S2Region derived class (our own, not from s2) I know faster way to compute covering. https://github.com/arangodb/arangodb/blob/d37f3d17f69dd388464cd509b749c771f01e1c55/arangod/IResearch/GeoAnalyzer.cpp#L389

> Could you get the options and make a coverer from that? Of course I can, I send code where I do it. But it's worse, instead of using internal...

> Aren't we really just talking about the difference between something(indexer.coverer()) and something(S2RegionCoverer(indexer.options())) @jmr Not really? Because get covering is not const. My main example is "set" of polylines. It's...

> In general if it will be possible I think better to have ability to create Indexer without coverer It can be done with making S2RegionIndexer template (implementation still can...

> This sounds like it might be a good option. Most of the implementations should be simple. Why do you need a shape index? How to implement MayIntersect good for...

It's linear

Hmm, maybe it's ok, because S2Polyline:: MayIntersects also linear. Btw what about set of points (GeoJson multipoints), I think creating cells from points and then Canonicalize it better than run...

> It will be helpful you explain what you're trying to do, what alternatives you considered, and why this is the best alternative Or you can just make termindexer more...

@jmr What do you think about separation to two classes, one derived another?