Problem/Motivation

AI Search has been split from AI: https://www.drupal.org/project/ai_search
AI has removed a bunch from base: https://www.drupal.org/project/ai/issues/3539214

Issue fork ai_vdb_provider_postgres-3563773

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

b_sharpe created an issue. See original summary.

eworwa made their first commit to this issue’s fork.

eworwa opened merge request !19

eworwa’s picture

eworwa commented
Status: Active » Needs review

Created MR !1 with AI Search 2.0 support.

This implements all required changes from the migration guide:
- Dependency updates to AI 2.0.x-dev and AI Search 1.0.x-dev
- Parent class migration to SearchApiAiVdbProviderBase
- Updated service dependencies
- Support for excluded entity IDs filter for better performance
- Maintained Key module integration for password handling

Ready for review.

scott_euser’s picture

scott_euser commented
Status: Needs review » Needs work

Thanks! I haven't tested this as I don't use Postgres as a VDB Provider myself, but left a couple comments on the MR. I'll make this more clear in the upgrade notes

scott_euser’s picture

scott_euser commented

Ah its already saying 2.0.x in https://www.drupal.org/project/ai_search/issues/3485449

scott_euser’s picture

scott_euser commented

Probably this should be 2.0.x branch of this module as well so 1.0.x branch can remain as targeting 1.x of AI Core

eworwa’s picture

eworwa commented
Status: Needs work » Needs review

Thanks @scott_euser, I implemented your suggestions to use ai_search ^2.0@alpha instead of dev version, since 2.0.0-alpha1 is available.

Regarding

Probably this should be 2.0.x branch of this module as well so 1.0.x branch can remain as targeting 1.x of AI Core

I agree, thou I'm not sure I have the right permission to create the 2.0.x branch or this is something you need to do at your end?