GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — bug: Incorrect Qdrant Online Store Path in repo_config.py · Issue #5206 · feast-dev/feast

bug: Incorrect Qdrant Online Store Path in repo_config.py #5206

New issue
Copy link
New issue
Copy link
Closed
#5207
Closed
bug: Incorrect Qdrant Online Store Path in repo_config.py#5206
#5207
Copy link
Labels
kind/bugpriority/p2

Description

@YassinNouh21
YassinNouh21
opened on Apr 1, 2025
Issue body actions

Description

The repo_config.py file contains an incorrect path for the Qdrant online store in the LEGACY_ONLINE_STORE_CLASS_FOR_TYPE and also in ONLINE_STORE_CLASS_FOR_TYPE dictionary. The current path points to a non-existent module:

"feast.infra.online_stores.contrib.qdrant.QdrantOnlineStore": "feast.infra.online_stores.cqdrant.QdrantOnlineStore",

This causes issues when trying to use the Qdrant online store with legacy configurations.

Current Behavior

When using a legacy configuration that references the Qdrant online store, the system fails to find the correct module path, resulting in import errors.

Expected Behavior

The path should point to the correct Qdrant online store module:

"feast.infra.online_stores.contrib.qdrant.QdrantOnlineStore": "feast.infra.online_stores.qdrant_online_store.qdrant.QdrantOnlineStore",

Activity

  1. YassinNouh21
    added
    kind/bug
    priority/p2
    on Apr 1, 2025
  2. YassinNouh21 commented on Apr 1, 2025

    @YassinNouh21
    YassinNouh21
    on Apr 1, 2025
    CollaboratorAuthor
    More actions

    @franciscojavierarceo I think it will be relating for testing the test_retrieve_online_qdrant_documents_v2

    Reacted by Francisco Javier Arceo
  3. YassinNouh21
    added 2 commits that reference this issue on Apr 1, 2025

    fix: Update Qdrant online store paths in repo_config.py

    This issue will close once commit d31e789 is merged into the 'master' branch.
    d31e789

    fix: Update Qdrant online store paths in repo_config.py

    This issue will close once commit 094a001 is merged into the 'master' branch.
    094a001
  4. franciscojavierarceo
    added a commit that references this issue on Apr 2, 2025

    fix: Update Qdrant online store paths in repo_config.py (#5207)

    This issue will close once commit ab35b0b is merged into the 'master' branch.
    ab35b0b
  5. franciscojavierarceo
    closed this as completedin #5207on Apr 2, 2025
  6. franciscojavierarceo
    added a commit that references this issue on Apr 7, 2025

    chore(release): release 0.48.0

    8838320
  7. jfw-ppi
    added 2 commits that reference this issue on Jun 7, 2025

    fix: Update Qdrant online store paths in repo_config.py (feast-dev#5207)

    This issue will close once commit 7fb0f87 is merged into the 'master' branch.
    7fb0f87

    chore(release): release 0.48.0

    This issue will close once commit cc36081 is merged into the 'master' branch.
    cc36081
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    kind/bugpriority/p2

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.