Skip to content

Commit

Permalink
fix: remove temp logging from a closed issue.
Browse files Browse the repository at this point in the history
  • Loading branch information
iloveagent57 committed Oct 3, 2023
1 parent 3a04091 commit e5a5bcc
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 14 deletions.
6 changes: 0 additions & 6 deletions enterprise_catalog/apps/api/tasks.py
Original file line number Diff line number Diff line change
Expand Up @@ -839,12 +839,6 @@ def _get_algolia_products_for_batch(
)
num_content_metadata_indexed = 0
for metadata in content_metadata_to_index:
# TODO: remove when https://2u-internal.atlassian.net/browse/ENT-7458 is resolved
if 'GTx+MGT6203x' in content_key:
logger.info(
f'[ENT-7458] {content_key} will be added to Algolia index'
)

# Build all the algolia products for this single metadata record and append them to
# `algolia_products_by_object_id`. This function contains all the logic to create duplicate/segmented records
# with non-overlapping UUID list fields to keep the product size below a fixed limit controlled by
Expand Down
9 changes: 1 addition & 8 deletions enterprise_catalog/apps/catalog/algolia_utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -187,16 +187,9 @@ def deadline_passed_checker():
(run_is_hidden_checker, 'advertised course run is hidden'),
(no_owners_checker, 'no owners exist'),
):
# TODO: remove/simplify logging when https://2u-internal.atlassian.net/browse/ENT-7458 is resolved
should_not_index = should_not_index_function()
if 'GTx+MGT6203x' in course_metadata.content_key:
logger.info(
f'[ENT-7458] {course_metadata.content_key} {log_message} '
f'checker has should_not_index={should_not_index}'
)

if should_not_index:
logger.info(f'[ENT-7458] Should not index {course_metadata.content_key}, {log_message}')
logger.info(f'Not indexing {course_metadata.content_key}, reason: {log_message}')
return False

return True
Expand Down

0 comments on commit e5a5bcc

Please sign in to comment.