You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Is your feature request related to a problem? Please describe.
Having the create_project method on the Feast Client fail when a project exists is important, but its also a pain in development. It means one part of the code in a notebook should be skipped every time.
Describe the solution you'd like
It would be convenient to be able to run create_project multiple times without having the second call throw an exception.
On second thought, it might be better to use the alternative suggested above. So we could encourage users to almost always just use set_project. When they apply resources or run commands it will create the project if it doesnt exist. If it does exist it will try to use it.
My thoughts on this topic have evolved a little bit. Here is what I am thinking
Projects should be invisible to the user if they don't want to use it. Meaning they should not have to create a project nor set a project.
If a user interacts with Feast without supplying a project, they are automatically assumed to be in a default project. I'd like to propose default for this project name, following the lead of Kubernetes having a default namespace.
For the purposes of this specific issue, I think the scope is a little bit more limited.
Set project should configure the local project, but not setting the project should keep the project string set to default
If a user creates a resource and a project does not exist, then the project should automatically be created for them. So it should run the creation in the background.
Activity
woop commentedon Feb 2, 2020
On second thought, it might be better to use the alternative suggested above. So we could encourage users to almost always just use
set_project. When they apply resources or run commands it will create the project if it doesnt exist. If it does exist it will try to use it.Joostrothweiler commentedon Mar 8, 2020
@woop I would be happy to pick this up as a first issue.
I could change the signature to:
def set_project(self, project: str, create_if_not_exists : bool = True):Would you say the preferred default value of
create_if_not_existsisTrueorFalsein this case?woop commentedon Mar 10, 2020
It would be awesome if you could pick this up @Joostrothweiler!
My thoughts on this topic have evolved a little bit. Here is what I am thinking
createa project norseta project.defaultfor this project name, following the lead of Kubernetes having adefaultnamespace.For the purposes of this specific issue, I think the scope is a little bit more limited.
defaultHope that makes sense :)