Skip to content
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.

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

Google cloud datastore: Client.__init__() got an unexpected keyword argument 'database' #4314

Closed
jimcybersec opened this issue Jun 25, 2024 · 4 comments · Fixed by #4348
Closed

Comments

@jimcybersec
Copy link

Expected Behavior

Can read online features from a Google Cloud Datastore instance.

Current Behavior

In the Python SDK, FeatureStore class attempts to initialize a datastore Client by passing a 'database' argument. Feast's setup.py lower bounds google-cloud-datastore at 2.1.0 whereas the 'database' argument was not introduced to the datastore Client until version 2.16.0 of google-cloud-datastore: https://github.com/googleapis/python-datastore/blob/v2.16.0/google/cloud/datastore/client.py#L301

This results in error "Client.init() got an unexpected keyword argument 'database'" with google-cloud-datastore <2.16.0 installed.

Steps to reproduce

Attempt to read online features from a known Google Cloud Datastore instance with google-cloud-datastore <2.16.0 and Feast version 0.38. This problem appears to also exist in the latest Feast release.

Specifications

  • Version: 0.38
  • Platform: Python, Google Cloud Datastore
  • Subsystem:

Possible Solution

Increase google-cloud-datastore lower bound in setup.py to >=2.16.0.

@tokoko
Copy link
Collaborator

tokoko commented Jun 26, 2024

2.16.0 has been out almost a year, so increasing a lower bound sounds good. @jimcybersec Would you like to submit a PR?

@amirali-shfz
Copy link

any workarounds meanwhile?

@tokoko
Copy link
Collaborator

tokoko commented Jul 2, 2024

@amirali-shfz forcing google-cloud-datastore version to be >=2.16.0 is the workaround. Are you unable to do that?

@amirali-shfz
Copy link

I ended up just downgrading to 0.37.1 lol

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants