Ulysse Buonomo
Ulysse Buonomo
@keithdoggett I'd say we follow Geos version, and adapt the test to the version with a conditional. I feel like having these kind of conditions directly in the codebase may...
Damn, I'd love to say that we're ready to merge, but I think there's a leak: ``` require "get_process_mem" trap(:INFO){p GetProcessMem.new.mb} po=RGeo::Geos.factory.parse_wkt("POLYGON ((0 0, 0.5 0, 0.5 0.5, 0 0.5,...
@eis-ioki thanks for the feedback! Do you want to contribute on that topic?
I do not see JRuby in the github action [tests.yml](https://github.com/rgeo/activerecord-postgis-adapter/blob/master/.github/workflows/tests.yml). What is the GitHub Action you are referencing?
I would not act on it until there is a real need (e.g. some jruby user is complaining, or the jruby code is generating failures in the main codebase)
@cscherban I would use a tool to monitor where the performance hit is (e.g: in ruby? in the db?). If it is in production you could use a monitor like...
@cscherban any chance you found the culprit ?
I had a quick look at it, and indeed we do not have the adapter specific code to handle collation (e.g. `SchemaDumper#schema_collation` is not overriden). This should not be so...
@keithdoggett I'd refrain from extending ruby object with a gem that isn't updated since 2013 and has not much stars. In favor of closing. WDYT?
According to [CockroachDB documentation](https://www.cockroachlabs.com/docs/stable/hash-sharded-indexes), I'd say yes. I'd suggest trying and running the query with your setup to see if it's ok, the error is quite verbose and should help...