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

Get() header data type #72

Open
adam-26 opened this issue Oct 23, 2015 · 4 comments
Open

Get() header data type #72

adam-26 opened this issue Oct 23, 2015 · 4 comments
Assignees

Comments

@adam-26
Copy link

adam-26 commented Oct 23, 2015

Hi,
When returning data from a .get() request I need to identify if the data was .set() using a buffer or a string. I'm assuming I could access this information from the header? but it's not included with the get callback.

Without having to track the data types <-> keys in my application, could this be achieved if the header data was returned in the get() callback?

Thanks, Adam.

@alevy
Copy link
Member

alevy commented Oct 23, 2015

Well, part of the header is returned, specifically the flags part, which I think is exactly the thing you'd want to use for this. Unfortunately, there is not yet support for actually setting the flags.

This is mostly blocked on changing the format of arguments to MemJS operations, the only way to add it now is add it after the callback argument, which is different from the Node conventions.

If we got flags working, do you think that would solve your problem?

@adam-26
Copy link
Author

adam-26 commented Oct 25, 2015

Yes, i believe that would solve the problem. Do you think it would take long to implement (I'm not expecting you to do it anytime soon, I'm just curious if you think it would be much work)?

I don't see a problem defying the node conventions to maintain backwards compatibility. You could always release in tandem a 1.x version that adheres to node conventions?

@alevy
Copy link
Member

alevy commented Oct 26, 2015

I think it wouldn't be about at most a day's worth of work to support this.

@alevy alevy self-assigned this Oct 26, 2015
@dterei
Copy link
Collaborator

dterei commented May 21, 2016

Merging #87 should make this very easy.

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

No branches or pull requests

3 participants