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.

Sign up for GitHub

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

Jump to bottom

query in sync mode? #46

Closed
lavmax opened this issue Dec 26, 2010 · 4 comments
Closed

query in sync mode? #46

lavmax opened this issue Dec 26, 2010 · 4 comments

Comments

@lavmax
Copy link

lavmax commented Dec 26, 2010

Is is possible to run query in sync mode? Some tricks?
@felixge
Copy link
Collaborator

felixge commented Jan 3, 2011

No. Node.js does not have sync networking, that's the whole point : ).

@vitmalina
Copy link

The method querySync can be implemented by using execution blocking. If I implement it, do I have any changes to have my pull request merged? This will help escape callback-hell that you can see currently if you need to call several queries sequentially.

@dresende
Copy link
Collaborator

Don't nest your callbacks, split into different functions and probably files. You'll see you're not going to have more than 3 or 4 scope levels.

@vitmalina
Copy link

Thank you for your advice. I agree, it will make is simpler. But querySync still makes the logic much simpler.

myspar pushed a commit to myspar/mysql that referenced this issue Dec 21, 2023
@yosifkit
Merge pull request mysqljs#46 from infosiftr/read-datadir-from-config
bbc5c89 
Update entrypoints to read DATADIR from the MySQL configuration directly instead of assuming /var/lib/docker
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@felixge @dresende @lavmax @vitmalina