Skip to content

fix: Fix push sources and add docs / tests pushing via the python feature server #2561

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
feast-ci-bot merged 18 commits into from
Apr 19, 2022
Merged

fix: Fix push sources and add docs / tests pushing via the python feature server #2561

Changes from 1 commit
Commits
Show all changes
18 commits
Select commit Hold shift + click to select a range
93b165e
bug: Fixing push API endpoint to include a way to specify whether the…
adchia Apr 17, 2022
ddba206
fix
adchia Apr 17, 2022
23c9989
prune out unneeded fields in push source
adchia Apr 18, 2022
315f2d8
prune out unneeded fields in push source
adchia Apr 18, 2022
ec3d5c2
prune out unneeded fields in push source
adchia Apr 18, 2022
de905af
fix comment
adchia Apr 18, 2022
e6df1c3
fix comment
adchia Apr 18, 2022
ffc7436
fix comment
adchia Apr 18, 2022
42f6e27
fix comment
adchia Apr 18, 2022
5b37bb6
fix generator
adchia Apr 18, 2022
cf39fc3
add data source creator teardown
adchia Apr 18, 2022
ea6aaed
add data source creator teardown
adchia Apr 18, 2022
22dc1a4
update push source to alpha
adchia Apr 18, 2022
47c1697
lint
adchia Apr 18, 2022
1e3e5ff
lint
adchia Apr 18, 2022
152686e
lint
adchia Apr 18, 2022
05bc6a6
lint
adchia Apr 18, 2022
3964921
lint
adchia Apr 18, 2022
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
Next Next commit
add data source creator teardown 
Signed-off-by: Danny Chiao <danny@tecton.ai>
  • Loading branch information
@adchia
adchia committed Apr 18, 2022
commit ea6aaed906dabdadebc279bcd4238003dda9c3aa
5 changes: 4 additions & 1 deletion sdk/python/tests/integration/e2e/test_python_feature_server.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,8 @@ def test_get_online_features():
response = client.post(
"/get-online-features", data=json.dumps(request_data_dict)
)

# Check entities and features are present
parsed_response = json.loads(response.text)
assert "metadata" in parsed_response
metadata = parsed_response["metadata"]
Expand All @@ -52,7 +54,6 @@ def test_get_online_features():
assert len(result["statuses"]) == 2 # Requested two entities
for status in result["statuses"]:
assert status == "PRESENT"

results_driver_id_index = response_feature_names.index("driver_id")
assert (
results[results_driver_id_index]["values"]
Expand All @@ -77,6 +78,8 @@ def test_push():
}
)
response = client.post("/push", data=json_data,)

# Check new pushed temperature is fetched
assert response.status_code == 200
assert get_temperatures(client, location_ids=[1]) == [initial_temp * 100]

Expand Down