-
Notifications
You must be signed in to change notification settings - Fork 993
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
fix: Initial commit targetting grpc registry server #4458
Conversation
Signed-off-by: Daniele Martinoli <[email protected]>
…east-dev#4465) introduced base class FeastError for all Feast exceptions, with initial methods to map the grpc and HTTP status code Signed-off-by: Daniele Martinoli <[email protected]>
Signed-off-by: Daniele Martinoli <[email protected]>
Signed-off-by: Daniele Martinoli <[email protected]>
@tokoko @tmihalac @lokeshrangineni ready for review now |
@dmartinol I think you need to rebase |
Signed-off-by: Daniele Martinoli <[email protected]>
Signed-off-by: Daniele Martinoli <[email protected]>
Signed-off-by: Daniele Martinoli <[email protected]>
@tokoko is this a known IT failure reason?
|
I'll force a retry, but that looks like a genuine credentials error |
sdk/python/feast/errors.py
Outdated
|
||
@staticmethod | ||
def from_error_detail(detail: str) -> Optional["FeastError"]: | ||
import importlib |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These are standard lib libraries, why not import top-level?
Signed-off-by: Daniele Martinoli <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
What this PR does / why we need it:
Intercept registry server failures and adds an additional message to the response with the failure status and details.
The proxy client then rebuilds the original error from the status data.
Which issue(s) this PR fixes:
Partially fixes #4392 for Registry Server
Misc
Need to add some UT/IT to prove the effectivness