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

Some qustion with the class 'SessionPool' #531

Closed
hesy007 opened this issue Jul 31, 2023 · 1 comment
Closed

Some qustion with the class 'SessionPool' #531

hesy007 opened this issue Jul 31, 2023 · 1 comment
Labels
type/question Type: question about the product

Comments

@hesy007
Copy link

hesy007 commented Jul 31, 2023

  1. Why add syncronized to the method public synchronized ResultSet execute(String stmt, Map<String, Object> parameterMap)?
  2. and why this method has been marked as 'deprecated' ?
@QingZ11 QingZ11 added the type/question Type: question about the product label Sep 5, 2023
@Nicole00
Copy link
Contributor

Nicole00 commented Dec 7, 2023

the synchronized lock is unnecessary, it has been removed.
For sessionPool, our original intention is to provide a thread-safe execution entry that is convenient for users to use for multi-threads. So we plan to only provide the execute interface and not encapsulate other interfaces.

@QingZ11 QingZ11 closed this as completed Dec 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/question Type: question about the product
Projects
None yet
Development

No branches or pull requests

3 participants