Skip to content

Commit

Permalink
Remove logic to change port to 8002 for prod data refresh. We'll inst…
Browse files Browse the repository at this point in the history
…ead change the port in prod to match local (#5111)
  • Loading branch information
stacimc authored Oct 29, 2024
1 parent c4fa959 commit 381ccee
Showing 1 changed file with 1 addition and 7 deletions.
8 changes: 1 addition & 7 deletions catalog/dags/data_refresh/distributed_reindex.py
Original file line number Diff line number Diff line change
Expand Up @@ -261,15 +261,9 @@ def create_connection(
"""
worker_conn_id = f"indexer_worker_{instance_id or media_type}"

# TODO: Locally we use 8003 to avoid collision with the legacy indexer worker that's
# part of the ingestion server. When the ingestion server and legacy data refresh are
# removed, we should modify the local environment to use 8002 here as well and
# avoid having to do this check.
port = "8003" if environment != PRODUCTION else "8002"

# Create the Connection
logger.info(f"Creating connection with id {worker_conn_id}")
worker_conn = Connection(conn_id=worker_conn_id, uri=f"http://{server}:{port}/")
worker_conn = Connection(conn_id=worker_conn_id, uri=f"http://{server}:8003/")
session = settings.Session()
session.add(worker_conn)
session.commit()
Expand Down

0 comments on commit 381ccee

Please sign in to comment.