Skip to content

Commit

Permalink
Merge pull request #1666 from CartoDB/feature/ch91890/sociallydetermi…
Browse files Browse the repository at this point in the history
…ned-onprem-user-can-t-geocode

Removed the batch_size parameter in the call to bulk_geocode
  • Loading branch information
antoniocarlon authored Jul 21, 2020
2 parents 05a5847 + a6de8ef commit a5151cb
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
3 changes: 0 additions & 3 deletions cartoframes/data/services/utils/geocoding_constants.py
Original file line number Diff line number Diff line change
@@ -1,6 +1,5 @@
__all__ = [
'HASH_COLUMN',
'BATCH_SIZE',
'DEFAULT_STATUS',
'QUOTA_SERVICE',
'STATUS_FIELDS',
Expand All @@ -12,8 +11,6 @@

HASH_COLUMN = 'carto_geocode_hash'

BATCH_SIZE = 200

DEFAULT_STATUS = {'gc_status_rel': 'relevance'}

QUOTA_SERVICE = 'hires_geocoder'
Expand Down
6 changes: 2 additions & 4 deletions cartoframes/data/services/utils/geocoding_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -144,16 +144,14 @@ def geocode_query(table, schema, street, city, state, country, status):
{street},
{city},
{state},
{country},
{batch_size}
{country}
)
""".format(
query=query,
street=column_name(street),
city=column_name(city),
state=column_name(state),
country=column_name(country),
batch_size=geocoding_constants.BATCH_SIZE
country=column_name(country)
)

status_assignment, status_columns = status_assignment_columns(status)
Expand Down

0 comments on commit a5151cb

Please sign in to comment.