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

Different namespaces used for memcache and datastore #42

Closed
luna-duclos opened this issue Aug 3, 2015 · 4 comments
Closed

Different namespaces used for memcache and datastore #42

luna-duclos opened this issue Aug 3, 2015 · 4 comments
Assignees
Labels

Comments

@luna-duclos
Copy link

If ctx and key have a different namespace for any kind of request in nds, then nds will use the key's namespace for the datastore request, as intended, but will do the memcache requests without creating a new context with that namespace in it, causing the memcache requests to be against a different namespace

@luna-duclos
Copy link
Author

Just pulled this and tested, my unit tests pass now, ty! :)

@jongillham
Copy link
Member

Thanks for reporting it!

@jongillham
Copy link
Member

@PSG-Luna just to let you know that I found an edge case bug with my namespace fix and have updated the repo with a fix b6608fb.

@luna-duclos
Copy link
Author

Thanks for the headsup!

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

No branches or pull requests

2 participants