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

Make query processor more friendly for embedded usage #57

Open
HouzuoGuo opened this issue Jun 2, 2014 · 4 comments
Open

Make query processor more friendly for embedded usage #57

HouzuoGuo opened this issue Jun 2, 2014 · 4 comments
Labels

Comments

@HouzuoGuo
Copy link
Owner

Currently, the query processor assumes input to be coming from deserialized JSON, therefore some query options such as limit and int-from are expected to be of type float64, which is awkward in embedded usage scenarios.

Change the behaviour so that during query processing, int, int64, uint64 are acceptable when JSON integer is expected.

@HouzuoGuo HouzuoGuo added the bug label Jun 2, 2014
@jameycribbs
Copy link

If you could also include an example in example.go of doing an embedded query without json that would be awesome. I am trying to figure out how to do an intersection query without first having to marshal to and then unmarshal from json. I am very new to go, so I know that is a big part of the problem, but if there was an example of what an intersection query looked like using native go structures, rather than json, it would be greatly appreciated.

@HouzuoGuo
Copy link
Owner Author

thanks very much & will do

@archey
Copy link
Collaborator

archey commented Feb 12, 2017

@HouzuoGuo maybe this one can be solved now as well? So it can be fixed for good

@HouzuoGuo
Copy link
Owner Author

@archey Good idea, but at the moment I'm working on some features in https://github.com/HouzuoGuo/websh. Are you feeling adventurous to add those examples to example.go?

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

3 participants