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

Updated dependencies versions #3

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

morazow
Copy link

@morazow morazow commented Feb 23, 2022

Fixes #1
Fixes #2

@morazow
Copy link
Author

morazow commented Feb 23, 2022

Please let me know if there are any suggestions.

@littleK0i
Copy link
Contributor

@morazow , thank you for your interest and contributions.

Since technically I am not working for badoo anymore, I am currently not in position to approve or reject changes in badoo organization. What I suggest to do is the following:

  1. Fork this repo and play around with it for a few weeks. Make it compatible with latest Kotlin and JVM trends, etc. Fix tests.
  2. Make one combined pull request.
  3. I'll ping a person who still works for badoo and who will be able to review & approve changes.

Please note, Exasol made a lot of changes in the most recent versions. For example, now the latest protocol version is 3. Exasol SAAS uses OpenID tokens for authentication. Encryption is now enabled by default and it will be mandatory starting from Exasol v8.

It might be worth sticking with JDBC for production-critical workloads. The only real advantage of kexasol is "CSV streaming".

@morazow
Copy link
Author

morazow commented Feb 24, 2022

Hey @littleK0i,

Thanks for the update! I have added changes for other two issues, I'll check issues with build.

It would be good to have continued maintenance and release of the library. At the moment, we are going to use it for a connector in enclosed cloud service. As you said we would like to use the CSV features from Java and JVM languages.

I'll discuss with our PM if joint collaboration could be arranged. Thanks a lot for the great work!

@littleK0i
Copy link
Contributor

Ah, you're from Exasol dev team.

In theory, you may just fork this repo and maintain it in Exasol organization. Since the license is Apache 2.0, no one will object to this.

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

Successfully merging this pull request may close these issues.

Add a convenience constructor for ExaConnectionOptions Change enum package name
2 participants