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

[NETBEANS-2418][NETBEANS-2298] update suggested MySQL Driver to latest #1202

Closed
wants to merge 362 commits into from
Closed

[NETBEANS-2418][NETBEANS-2298] update suggested MySQL Driver to latest #1202

wants to merge 362 commits into from

Conversation

mcdonnell-john
Copy link
Contributor

There was a defect related to the JDBC Connection String for MySQL, I took a look and noticed we were suggesting an older driver.

I've updated the suggested driver to the latest, and ran some CRUD DB tests with it with NB 11.0 with no issues

@mcdonnell-john
Copy link
Contributor Author

Not sure, but looking at Travis:
The Checks fail on the check: RUN_TESTS_JDK9PLUS

But theres no errors and the end of the logs say:
"The job exceeded the maximum log length, and has been terminated."

Are we running too many checks in these Travis builds?

@wadechandler
Copy link
Member

Could be John. The changes themselves look just fine. Will check the mailing lists to see if this is a known thing too. On mobile right now.

@matthiasblaesing
Copy link
Contributor

One think that the driver reminded me at work: The driver class was moved to com.mysql.cj.jdbc.Driver, should that also be updated?

@mcdonnell-john
Copy link
Contributor Author

I didn't come across a change to the class when testing but will double check before merging to be sure

@wadechandler
Copy link
Member

wadechandler commented Apr 23, 2019

It is more likely a deprecation warning https://dev.mysql.com/doc/connector-j/8.0/en/connector-j-api-changes.html

Notice the old class name is deprecated. I would not expect tests to fail, but it makes sense to update.

Jaroslav Tulach and others added 25 commits May 24, 2019 09:10
[NETBEANS-2573] Pass files to test in env variable
Allow any kind of Mode to be created programatically from a ModeConfig XML String so that a TopComponent may then be programatically docked into that Mode.
…st be found (message + stacktrace)

- stacktrace may be empty
[NETBEANS-1638] export org.apache.felix.resolver.reason
 [NETBEANS-1851] PHPUnit java.lang.AssertionError: At least 2 lines must be found (message + stacktrace)
NETBEANS-2565: ide.ks file tracked again as part of updatecenters module
[NETBEANS-2361] Fix app.conf and launcher on macOS after changes in #573
One of those deprecated warnings is "getURL() in FileObject has been deprecated".
This work is simply to remove these warnings and make the build less noisy.
matthiasblaesing and others added 27 commits July 25, 2019 22:20
[NETBEANS-2783] Fix missing dependencies nodes in Project view when editing POM file
…ts. (#1329)

* [NETBEANS-2729] Let's development and release snap descriptors co-exists
Update htmlparser/lexer to support html5 attribute syntax
Fix ClassPathTest in ide/api.java.classpath module
[NETBEANS-2881] Prevent java.lang.StringIndexOutOfBoundsException
…on of javax.lang.model.element.ModuleElement, when the available javac is not new enough.
…e redirecting / release templating

Re-add special 3rd-party UC and 8.2 centre while still required in releases
…-brackets

[NETBEANS-2910] Suggest using brackets for accessing arrays instead of curly braces
The references issue found cases, where the CSS grammar lead to
an inifite backtracking loop. This patch works around that limitation
by requireing that the parsing progresses.

The implementation counts the read calls issued without progressing
through the stream. If the read count withouth progress exceeds
10.000.000 reads, parsing is stopped.

The number was found experimentally - it needs to be large enough to
handle real world cases and low enough that progress happens early
enough.
[NETBEANS-1047] Don't add whitespace to a wrong place when textual operators are used
- Allow empty level option to use a level of a configuration file
[NETBEANS-2974] PHPStan - make level optional
@mcdonnell-john
Copy link
Contributor Author

Closing due to re-basing issue

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

Successfully merging this pull request may close these issues.

None yet