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

Proposed changes #2

Open
wdavidw opened this issue Nov 4, 2011 · 0 comments
Open

Proposed changes #2

wdavidw opened this issue Nov 4, 2011 · 0 comments

Comments

@wdavidw
Copy link

wdavidw commented Nov 4, 2011

Hi,

After using, i encountered a few concerns which i wanted to share with you. For illustration, i just pushed my code to "https://github.com/wdavidw/node-hive":

  • Creating client each time the execute method is called didn't work for me. For exemple running use my_db, set my.property=value, INSERT ... need to be run inside the same connection (or client)
  • Move generated Thrift code from node_module to src
  • Added multi version layout/support
  • Use NPM to load the Thrift module
  • API with only query, execute and end function similar to yours

Currently, data parsing after executing a query is simpler than yours since i dont retrieve the schema but this could be added.

Are you interested into merging the 2 modules? i would take care of everything but that implies a breaking of your API/behavior. Otherwise i'll publish it separately. I leave the decision up to you.

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

No branches or pull requests

1 participant