Skip to content

fix: Removed time.sleep(4) to avoid unnecessary delay in the script#159

Merged
Roopan-Microsoft merged 1 commit intodevfrom
psl-remove-sleeptime-index
Feb 5, 2025
Merged

fix: Removed time.sleep(4) to avoid unnecessary delay in the script#159
Roopan-Microsoft merged 1 commit intodevfrom
psl-remove-sleeptime-index

Conversation

@Kanchan-Microsoft
Copy link
Contributor

@Kanchan-Microsoft Kanchan-Microsoft commented Feb 5, 2025

Purpose

  • This pull request includes a small change to the Deployment/scripts/index_scripts/process_data.py file. The change removes an unnecessary sleep call from the get_details function to improve the script's efficiency.

  • Deployment/scripts/index_scripts/process_data.py: Removed time.sleep(4) to avoid unnecessary delay in the script execution.

  • Removed the sleep time for KM generic deployment

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • Yes

Other Information

Removed the sleep time
@Roopan-Microsoft Roopan-Microsoft changed the title fix: Psl remove sleeptime index fix: Removed time.sleep(4) to avoid unnecessary delay in the script Feb 5, 2025
@Roopan-Microsoft Roopan-Microsoft merged commit 33798c6 into dev Feb 5, 2025
8 checks passed
@Avijit-Microsoft Avijit-Microsoft deleted the psl-remove-sleeptime-index branch March 28, 2025 05:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants