censusreporter / census-postgres-scripts

Scripts used to set up census-postgres on an Amazon EC2 instance.
MIT License
65 stars 34 forks source link

Why insert NYC separately to census_name_lookup? #4

Closed JoeGermuska closed 10 years ago

JoeGermuska commented 10 years ago

The null geometry inserted here https://github.com/censusreporter/census-postgres-scripts/blob/master/13_index_tiger_2012.sql#L82-L83

led to a bug. The bug was fixed by both manually deleting the row above from the production DB and by implementing a null check. But why is this here? We may want to double check the assumption that it's safe to tweak census_name_lookup.

iandees commented 10 years ago

It's there to support the "new york city" search. We use elasticsearch+synonyms for that nowadays though.