Skip to content
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

Un-necessary warning message with each requests to the get_online_features() API function. #4591

Open
Atul-Anand-Jha opened this issue Oct 1, 2024 · 0 comments

Comments

@Atul-Anand-Jha
Copy link

Expected Behaviour

Ideally, It should not throw any warning message. Since, the function under examination is an internal function.

Current Behaviour

After upgrading the feast from v0.36.0 to v0.40.1; and shifting from _get_online_features() (internal API) to get_online_features() (public API). FYI, _get_online_features() private API has been deprecated with feast v0.38.*

I am getting this warning message on each request i hit to my feature Store. I am using Python-SDK for feast feature-store client.

Warning message -

2024-10-01 13:36:39.126 IST
_list_feature_views will make breaking changes. Please use _list_batch_feature_views instead. _list_feature_views will behave like _list_all_feature_views in the future.

Extended log -

sourceLocation: {
file: "/usr/local/lib/python3.9/site-packages/feast/utils.py"
function: "_list_feature_views"
line: "750"
}
textPayload: "_list_feature_views will make breaking changes. Please use _list_batch_feature_views instead. _list_feature_views will behave like _list_all_feature_views in the future."
timestamp: "2024-10-01T08:06:39.126157964Z"
traceSampled: false
}

ref Image -
Screenshot 2024-10-01 at 2 10 00 PM

This is annoying and producing unwanted logs with each requests.

Steps to reproduce

Upgrade Feast version to v0.40.1 and use the public API (get_online_features()) to fetch the features from online Store (REDIS).

Specifications

  • Version: v0.40.1
  • Platform: GKE
  • Subsystem: Python3.9.12, requests - v2.28.1

Possible Solution

Disable this warning, or update the abstract method calling internally. Since, the functions -
"_list_feature_views", "_list_batch_feature_views", "_list_all_feature_views" are all internal functions, and can't be controlled by end-users using feast's public API.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant