[jdbc-v2] Fixes in the PreparedStatementImpl class, the initial length of the parameters is incorrectly changed in the clearParameters method

[CCweixiao]

Summary

Closes #2299

Checklist

Delete items not relevant to your PR:

• Closes #
• Unit and integration tests covering the common scenarios were added
• A human-readable description of the changes was provided to include in CHANGELOG
• For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ chernser
❌ jielongping

---

jielongping seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[chernser]

Good day, @CCweixiao!
Thank you for the contribution!
Would you please add at least one test to verify the problem is fixed?

Thanks!

[CCweixiao]

Good day, @CCweixiao! Thank you for the contribution! Would you please add at least one test to verify the problem is fixed?

Thanks!

Thanks a lot for the code review, I've submitted a test case for the corresponding code fix, please help me review again. @chernser

[chernser]

this is not a good practice - if someone change an original method then test will not be verifying a production code.
Please just make compileSql package private. This is ok because tests sometimes need to do so.

[chernser]

please remove this class.

[chernser]

@CCweixiao thanks!
I've reviewed - please see my comments.

[CCweixiao]

@CCweixiao thanks! I've reviewed - please see my comments.

Your suggestion was taken and the test case modifications were completed @chernser

[mzitnik]

@CCweixiao can you sign the CLA

[CCweixiao]

@CCweixiao can you sign the CLA

[chernser]

Good day, @CCweixiao !

Would you please try signing CLA once again?
Sorry for asking after so long time.

Thanks!

[CCweixiao]

Good day, @CCweixiao !

Would you please try signing CLA once again? Sorry for asking after so long time.

Thanks!

It seems I need to change my commit username and email. I didn't pay attention when submitting before. How should I handle this? Should I bring up a new PR?

[chernser]

Yes,
I think new PR will trigger same check and we do not need to merge it - just simple change.

Thank you!