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 — feat: Implement and enhance remote document retrieval functionality by jyejare · Pull Request #5487 · feast-dev/feast

Conversation

@jyejare

@jyejare jyejare commented Jul 1, 2025

Copy link
Copy Markdown
Collaborator

PR includes:

  1. Implements a new request model for remotely retrieving online documents
  2. Updates the feature server to handle document retrieval,
  3. Both V1 and V2 docvuments retrival is implemented
  4. Unit tests to ensure functionality.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
franciscojavierarceo reacted with hooray emoji franciscojavierarceo reacted with heart emoji
All reactions
  • 🎉 1 reaction
  • ❤️ 1 reaction
@jyejare
jyejare requested a review from a team as a code owner July 1, 2025 09:56
@jyejare jyejare changed the title Enhance remote document retrieval functionality Implement and enhance remote document retrieval functionality Jul 1, 2025
@jyejare jyejare changed the title Implement and enhance remote document retrieval functionality feat: Implement and enhance remote document retrieval functionality Jul 1, 2025
@jyejare
jyejare force-pushed the remote_retrieve_docs branch from 7d92b04 to 48f27f7 Compare July 1, 2025 10:42
@ntkathole ntkathole added the ok-to-test label Jul 1, 2025
@jyejare
jyejare force-pushed the remote_retrieve_docs branch from 48f27f7 to eb77f31 Compare July 1, 2025 13:31
ntkathole
ntkathole reviewed Jul 9, 2025
View reviewed changes
Comment thread sdk/python/feast/feature_server.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
ntkathole
ntkathole reviewed Jul 9, 2025
View reviewed changes
Comment thread sdk/python/feast/infra/online_stores/remote.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
franciscojavierarceo
franciscojavierarceo requested changes Jul 10, 2025
View reviewed changes
Comment thread sdk/python/feast/feature_server.py Resolved

Uh oh!

There was an error while loading. Please reload this page.
jyejare added 6 commits July 24, 2025 20:59
@jyejare
Remotely retrieve the docs
7c951eb 
Signed-off-by: jyejare <[email protected]>
@jyejare
Feature server rectified for documents retrival
7839fc5 
Signed-off-by: jyejare <[email protected]>
@jyejare
Updates to remote retrival for response extraction
18bfcf4 
Signed-off-by: jyejare <[email protected]>
@jyejare
Remote retrive online doc v2
2043b09 
Signed-off-by: jyejare <[email protected]>
@jyejare
Unit tests for Remote docuemnts retrival
8bcb2ae 
Signed-off-by: jyejare <[email protected]>
@jyejare
Fixed checks and comments
dfafa56 
Signed-off-by: jyejare <[email protected]>
@jyejare
jyejare force-pushed the remote_retrieve_docs branch from eb77f31 to dfafa56 Compare July 24, 2025 15:31
@jyejare jyejare mentioned this pull request Jul 24, 2025
Consolidate document retrieval methods: Remove v1 and standardize on v2 #5537
Open
17 tasks
franciscojavierarceo
franciscojavierarceo approved these changes Jul 28, 2025
View reviewed changes
@franciscojavierarceo
franciscojavierarceo merged commit d095b96 into feast-dev:master Jul 28, 2025
haoxu0 pushed a commit that referenced this pull request Aug 11, 2025
@jyejare @haoxu0
feat: Implement and enhance remote document retrieval functionality (#…
9f44ef7 
…5487)

* Remotely retrieve the docs

Signed-off-by: jyejare <[email protected]>

* Feature server rectified for documents retrival

Signed-off-by: jyejare <[email protected]>

* Updates to remote retrival for response extraction

Signed-off-by: jyejare <[email protected]>

* Remote retrive online doc v2

Signed-off-by: jyejare <[email protected]>

* Unit tests for Remote docuemnts retrival

Signed-off-by: jyejare <[email protected]>

* Fixed checks and comments

Signed-off-by: jyejare <[email protected]>

---------

Signed-off-by: jyejare <[email protected]>
franciscojavierarceo pushed a commit that referenced this pull request Aug 14, 2025
@feast-ci-bot
chore(release): release 0.52.0
acb97fb 
# [0.52.0](v0.51.0...v0.52.0) (2025-08-14)

### Bug Fixes

* Correct entity value type mapping for aliased feature views ([#5492](#5492)) ([bdf20bb](bdf20bb))
* Correct namespace reference in remote Feast project setup for operator upgrade and previous version tests ([df391ec](df391ec))
* dell pydantic v1 ([1189512](1189512))
* Fixed the entity to on-demand feature view relationship ([1c59bba](1c59bba))
* Make transformers optional ([#5544](#5544)) ([a4eef38](a4eef38))
* Push Source inherits the timestamp fields from Data Source ([#5550](#5550)) ([b7ea5cc](b7ea5cc))
* Remove the devcontainer folder. ([a9815c2](a9815c2))

### Features

* Added API for discovering Feature Views by popular tags ([#5558](#5558)) ([2e5f564](2e5f564))
* Added filtering support for featureView and featureServices api ([#5552](#5552)) ([897b3f3](897b3f3))
* Added global search api and necessary unit tests ([#5532](#5532)) ([dd3061f](dd3061f))
* Added Ray Compute Engine and Ray Offline Store Support ([#5526](#5526)) ([72de088](72de088))
* Added recent visit logging api for registry server ([#5545](#5545)) ([2adcf2c](2adcf2c))
* **auth:** support client-credentials & static token for OIDC client auth ([fc44222](fc44222))
* **auth:** support client-credentials & static token for OIDC client auth ([795fc06](795fc06))
* Implement and enhance remote document retrieval functionality ([#5487](#5487)) ([d095b96](d095b96))
* Implemented consistent error handling ([7f10151](7f10151))
* Offline Store historical features retrieval without entity df, but based on datatime range ([#5527](#5527)) ([df942b9](df942b9))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ntkathole ntkathole ntkathole left review comments
@franciscojavierarceo franciscojavierarceo franciscojavierarceo approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

ok-to-test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@jyejare @franciscojavierarceo @ntkathole
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402520 — 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.