check last_pruned instead of is_pruning

[rkuo-danswer]

Description

Potential workaround for race condition in wait for prune and some buggy blocking behavior in default HTTPServer.

How Has This Been Tested?

[Describe the tests you ran to verify your changes]

Accepted Risk

[Any know risks or failure modes to point out to reviewers]

Related Issue(s)

[If applicable, link to the issue(s) this PR addresses]

Checklist:

• All of the automated tests pass
• All PR comments are addressed and marked resolved
• If there are migrations, they have been rebased to latest main
• If there are new dependencies, they are added to the requirements
• If there are new environment variables, they are added to all of the deployment methods
• If there are new APIs that don't require auth, they are added to PUBLIC_ENDPOINT_SPECS
• Docker images build and basic functionalities work
• Author has done a final read through of the PR right before merge

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
internal-search	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Oct 16, 2024 6:35pm

[greptile-apps]

PR Summary

This pull request modifies the pruning process for connector-credential pairs, focusing on using a 'last_pruned' timestamp instead of an 'is_pruning' boolean to address potential race conditions and improve concurrent request handling.

• Updated backend/danswer/server/documents/cc_pair.py to return the last pruning timestamp instead of checking if pruning is in progress
• Modified backend/tests/integration/common_utils/managers/cc_pair.py to use 'last_pruned' timestamp in CCPairManager for pruning status checks
• Changed backend/tests/integration/tests/pruning/test_pruning.py to use ThreadingHTTPServer for improved concurrent request handling
• Adjusted backend/tests/integration/connector_job_tests/slack/test_prune.py to capture a timestamp before pruning for more accurate status checks

_{4 file(s) reviewed, 1 comment(s)
Edit PR Review Bot Settings}