On Tuesday, June 24, 2014 3:32:41 AM UTC-4, Damian Dimmich wrote:

I intend to develop further syntax/query support for this type on an as 
> needed basis for now - suggestions and comments are much appreciated.
>

1. Nice work!  Now I'm more likely to use jsonb!

2. I ran into some sqlalchemy issues with postgresql and gin indexes 
recently.  the default `.match()` didn't accept the regconfig argument, 
which can be a necessary argument to properly hint the query-planner to use 
the index.  the result was the index is ignored and the column was being 
cast into ts_vector in real-time.  on a few thousand rows, there was a 100x 
difference in query time.    ( discussion here : 
https://bitbucket.org/zzzeek/sqlalchemy/issue/3078/postgresql-full-text-search-on-match
 
; there's a PR in the queue too )

so my suggestion would be to make sure you run the results of the generated 
SQL through `EXPLAIN ANALYZE` on a decently large test dataset to make sure 
the gin indexes are being used properly.  You could end up the situation 
were the sql is totally valid and "looks right", but it's structured in a 
way that the indexes are ignored.

-- 
You received this message because you are subscribed to the Google Groups 
"sqlalchemy" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/sqlalchemy.
For more options, visit https://groups.google.com/d/optout.

Reply via email to