Skip to content

fix: Initial commit targetting grpc registry server #4458

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.

Sign up for GitHub

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

Merged
tokoko merged 10 commits into from
Aug 30, 2024
Merged

fix: Initial commit targetting grpc registry server #4458

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
3e07339
initial commit targetting grpc registry server
dmartinol Aug 28, 2024
e833ff1
refactor: Introduced base class FeastError for all Feast exceptions (…
dmartinol Aug 29, 2024
d94c9ac
initial commit targetting grpc registry server
dmartinol Aug 28, 2024
acd9763
fixed merge error
dmartinol Aug 29, 2024
592c1ce
initial commit targetting grpc registry server
dmartinol Aug 28, 2024
73070ca
fixed merge error
dmartinol Aug 29, 2024
24661df
integrated comment
dmartinol Aug 29, 2024
400615a
Merge branch 'master' into issue-4392
dmartinol Aug 29, 2024
618cb00
Merge branch 'feast-dev:master' into issue-4392
dmartinol Aug 30, 2024
50306c1
moved imports as per comment
dmartinol Aug 30, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
moved imports as per comment 
Signed-off-by: Daniele Martinoli <dmartino@redhat.com>
  • Loading branch information
@dmartinol
dmartinol committed Aug 30, 2024
commit 50306c1041cad50029f85f1ea9370a3c87ab8ba6
6 changes: 2 additions & 4 deletions sdk/python/feast/errors.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
import importlib
import json
import logging
from typing import Any, List, Optional, Set

Expand Down Expand Up @@ -36,7 +38,6 @@ def to_error_detail(self) -> str:
Returns:
str: a string representation of a JSON document including `module`, `class` and `message` fields.
"""
import json

m = {
"module": f"{type(self).__module__}",
Expand All @@ -47,9 +48,6 @@ def to_error_detail(self) -> str:

@staticmethod
def from_error_detail(detail: str) -> Optional["FeastError"]:
import importlib
import json

try:
m = json.loads(detail)
if all(f in m for f in ["module", "class", "message"]):
Expand Down