search

koopjs / koop-pgcache

PostGIS cache for Koop.
Other
4 stars 4 forks source link

remove unnecessary json parsing #29

Closed dmfenton closed 9 years ago

dmfenton commented 9 years ago

This PR changes DB select queries to request JSON instead of casting to string and then parsing. It would be nice to have some benchmarks to see how much this helps, but that's another project.

ungoldman commented 9 years ago

I've been seeing this off-by-one error pretty regularly with the pgcache travis tests, which will usually disappear after rerunning tests.

screen shot 2015-08-22 at 4 50 07 pm

@dmfenton any idea what might be causing it?

dmfenton commented 9 years ago

Fixed that problem in: https://github.com/koopjs/koop-pgcache/pull/30

ungoldman commented 9 years ago

@dmfenton you know how to do the rebase magic?

dmfenton commented 9 years ago

rebased