-
Notifications
You must be signed in to change notification settings - Fork 1.3k
FeatureStore._plan should be a public method #2895
Copy link
Copy link
Closed
Labels
Community Contribution NeededWe want community to contributeWe want community to contributearea/sdksgood first issueGood for newcomersGood for newcomerskind/featureNew feature or requestNew feature or request
Description
Metadata
Metadata
Assignees
Labels
Community Contribution NeededWe want community to contributeWe want community to contributearea/sdksgood first issueGood for newcomersGood for newcomerskind/featureNew feature or requestNew feature or request
Type
Fields
Give feedbackNo fields configured for issues without a type.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
https://github.com/feast-dev/feast/blob/master/sdk/python/feast/feature_store.py#L639
Ideally, all FeatureStore methods that back a CLI operation should have a public facing programatic API. In this case,
feast planusersFeatureStore._plan-> which signifies that it should be treated as a private method.Describe the solution you'd like
The change would be as simple as refactoring it to
FeatureStore.plan.