Skip to content

Commit

Permalink
feat: Add cli list/describe for SavedDatasets, StreamFeatureViews, & … (
Browse files Browse the repository at this point in the history
feast-dev#4487)

feat: Add cli list/describe for SavedDatasets, StreamFeatureViews, & ValidationReferences

Signed-off-by: Tommy Hughes <tohughes@redhat.com>
  • Loading branch information
tchughesiv authored Sep 6, 2024
1 parent 2118719 commit 7b250e5
Show file tree
Hide file tree
Showing 3 changed files with 183 additions and 0 deletions.
150 changes: 150 additions & 0 deletions sdk/python/feast/cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -470,6 +470,156 @@ def on_demand_feature_view_list(ctx: click.Context, tags: list[str]):
print(tabulate(table, headers=["NAME"], tablefmt="plain"))


@cli.group(name="saved-datasets")
def saved_datasets_cmd():
"""
[Experimental] Access saved datasets
"""
pass


@saved_datasets_cmd.command("describe")
@click.argument("name", type=click.STRING)
@click.pass_context
def saved_datasets_describe(ctx: click.Context, name: str):
"""
[Experimental] Describe a saved dataset
"""
store = create_feature_store(ctx)

try:
saved_dataset = store.get_saved_dataset(name)
except FeastObjectNotFoundException as e:
print(e)
exit(1)

print(
yaml.dump(
yaml.safe_load(str(saved_dataset)),
default_flow_style=False,
sort_keys=False,
)
)


@saved_datasets_cmd.command(name="list")
@tagsOption
@click.pass_context
def saved_datasets_list(ctx: click.Context, tags: list[str]):
"""
[Experimental] List all saved datasets
"""
store = create_feature_store(ctx)
table = []
tags_filter = utils.tags_list_to_dict(tags)
for saved_dataset in store.list_saved_datasets(tags=tags_filter):
table.append([saved_dataset.name])

from tabulate import tabulate

print(tabulate(table, headers=["NAME"], tablefmt="plain"))


@cli.group(name="stream-feature-views")
def stream_feature_views_cmd():
"""
[Experimental] Access stream feature views
"""
pass


@stream_feature_views_cmd.command("describe")
@click.argument("name", type=click.STRING)
@click.pass_context
def stream_feature_views_describe(ctx: click.Context, name: str):
"""
[Experimental] Describe a stream feature view
"""
store = create_feature_store(ctx)

try:
stream_feature_view = store.get_stream_feature_view(name)
except FeastObjectNotFoundException as e:
print(e)
exit(1)

print(
yaml.dump(
yaml.safe_load(str(stream_feature_view)),
default_flow_style=False,
sort_keys=False,
)
)


@stream_feature_views_cmd.command(name="list")
@tagsOption
@click.pass_context
def stream_feature_views_list(ctx: click.Context, tags: list[str]):
"""
[Experimental] List all stream feature views
"""
store = create_feature_store(ctx)
table = []
tags_filter = utils.tags_list_to_dict(tags)
for stream_feature_view in store.list_stream_feature_views(tags=tags_filter):
table.append([stream_feature_view.name])

from tabulate import tabulate

print(tabulate(table, headers=["NAME"], tablefmt="plain"))


@cli.group(name="validation-references")
def validation_references_cmd():
"""
[Experimental] Access validation references
"""
pass


@validation_references_cmd.command("describe")
@click.argument("name", type=click.STRING)
@click.pass_context
def validation_references_describe(ctx: click.Context, name: str):
"""
[Experimental] Describe a validation reference
"""
store = create_feature_store(ctx)

try:
validation_reference = store.get_validation_reference(name)
except FeastObjectNotFoundException as e:
print(e)
exit(1)

print(
yaml.dump(
yaml.safe_load(str(validation_reference)),
default_flow_style=False,
sort_keys=False,
)
)


@validation_references_cmd.command(name="list")
@tagsOption
@click.pass_context
def validation_references_list(ctx: click.Context, tags: list[str]):
"""
[Experimental] List all validation references
"""
store = create_feature_store(ctx)
table = []
tags_filter = utils.tags_list_to_dict(tags)
for validation_reference in store.list_validation_references(tags=tags_filter):
table.append([validation_reference.name])

from tabulate import tabulate

print(tabulate(table, headers=["NAME"], tablefmt="plain"))


@cli.command("plan", cls=NoOptionDefaultFormat)
@click.option(
"--skip-source-validation",
Expand Down
17 changes: 17 additions & 0 deletions sdk/python/tests/integration/offline_store/test_validation.py
Original file line number Diff line number Diff line change
Expand Up @@ -305,6 +305,23 @@ def test_e2e_validation_via_cli(environment, universal_data_sources):
assert p.returncode == 0, p.stderr.decode()
assert "Validation successful" in p.stdout.decode(), p.stderr.decode()

p = runner.run(
["saved-datasets", "describe", saved_dataset.name], cwd=local_repo.repo_path
)
assert p.returncode == 0, p.stderr.decode()

p = runner.run(
["validation-references", "describe", reference.name],
cwd=local_repo.repo_path,
)
assert p.returncode == 0, p.stderr.decode()

p = runner.run(
["feature-services", "describe", feature_service.name],
cwd=local_repo.repo_path,
)
assert p.returncode == 0, p.stderr.decode()

# make sure second validation will use cached profile
shutil.rmtree(saved_dataset.storage.file_options.uri)

Expand Down
16 changes: 16 additions & 0 deletions sdk/python/tests/integration/registration/test_universal_cli.py
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,12 @@ def test_universal_cli():
assertpy.assert_that(result.returncode).is_equal_to(0)
result = runner.run(["permissions", "list"], cwd=repo_path)
assertpy.assert_that(result.returncode).is_equal_to(0)
result = runner.run(["validation-references", "list"], cwd=repo_path)
assertpy.assert_that(result.returncode).is_equal_to(0)
result = runner.run(["stream-feature-views", "list"], cwd=repo_path)
assertpy.assert_that(result.returncode).is_equal_to(0)
result = runner.run(["saved-datasets", "list"], cwd=repo_path)
assertpy.assert_that(result.returncode).is_equal_to(0)

# entity & feature view describe commands should succeed when objects exist
result = runner.run(["entities", "describe", "driver"], cwd=repo_path)
Expand Down Expand Up @@ -95,6 +101,16 @@ def test_universal_cli():
assertpy.assert_that(result.returncode).is_equal_to(1)
result = runner.run(["permissions", "describe", "foo"], cwd=repo_path)
assertpy.assert_that(result.returncode).is_equal_to(1)
result = runner.run(
["validation-references", "describe", "foo"], cwd=repo_path
)
assertpy.assert_that(result.returncode).is_equal_to(1)
result = runner.run(
["stream-feature-views", "describe", "foo"], cwd=repo_path
)
assertpy.assert_that(result.returncode).is_equal_to(1)
result = runner.run(["saved-datasets", "describe", "foo"], cwd=repo_path)
assertpy.assert_that(result.returncode).is_equal_to(1)

# Doing another apply should be a no op, and should not cause errors
result = runner.run(["apply"], cwd=repo_path)
Expand Down

0 comments on commit 7b250e5

Please sign in to comment.