Skip to content

feat: Remove AI search service for PostgreSQL configuration to optimize deployment #1624

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jan 13, 2025

Conversation

Prasanjeet-Microsoft
Copy link
Contributor

Purpose

  • This PR aims to modify the deployment process so that the AI search service is excluded when the database type is set to PostgreSQL. This change optimizes the deployment by removing unnecessary services for PostgreSQL configurations

Does this introduce a breaking change?

  • Yes
  • No

How to Test

  • Get the code
git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

What to Check

Verify that the following are valid

  • Ensure that the AI search service is correctly excluded when the database type is PostgreSQL.
  • Verify that the PostgreSQL database configuration works as expected without the AI search service.
  • Confirm that other database types, if any, still include the AI search service as required.
  • Validate that the deployment process completes without errors.

Copy link

Coverage

Coverage Report •
FileStmtsMissCoverMissing
code/backend/batch/utilities/orchestrator
   orchestrator_base.py51198%34
TOTAL381985777% 

Tests Skipped Failures Errors Time
395 0 💤 0 ❌ 0 🔥 55.019s ⏱️

@Prajwal-Microsoft Prajwal-Microsoft merged commit 1f22786 into dev Jan 13, 2025
10 checks passed
@Prasanjeet-Microsoft Prasanjeet-Microsoft deleted the PSL-US-12236 branch January 29, 2025 05:53
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