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

Upgrade to Java 11 [HZ-3617] #289

Merged
merged 9 commits into from
Dec 18, 2023

Conversation

orcunc
Copy link
Contributor

@orcunc orcunc commented Dec 6, 2023

Upgrade to Java 11
Upgrade various maven plugins
Upgrade Mockio version
Upgrade Jupiter version
Fix warnings in unit tests

Jira : https://hazelcast.atlassian.net/browse/HZ-3617

@orcunc orcunc self-assigned this Dec 12, 2023
.gitattributes Outdated
Comment on lines 1 to 2
# linux line-endings
* text eol=lf
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This doesn't seem to be correct. Why force all files to be text? And why not allow git to convert text files to platform-specific line endings?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought having explicit configuration instead of relying on the user's git configuration would be better. User may not have configured his git for automatic line ending.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still think this is wrong. Windows users will see the sources with wrong file-endings in their working tree. At least it's how I understand the explanation in https://git-scm.com/docs/gitattributes

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reverted : de8569b

Copy link
Contributor

@frant-hartm frant-hartm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly LGTM.

SqlResult sqlResult = member.getSql()
.execute("create view emp_dept_view as select * from emp join dept on emp.age=dept.__key");
sqlResult.close();
Connection connection = DriverManager.getConnection(JDBC_HAZELCAST_LOCALHOST);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this connection closed?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rev : 9a6f973

@orcunc orcunc requested a review from kwart December 14, 2023 11:37
.gitattributes Outdated
Comment on lines 1 to 2
# linux line-endings
* text eol=lf
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I still think this is wrong. Windows users will see the sources with wrong file-endings in their working tree. At least it's how I understand the explanation in https://git-scm.com/docs/gitattributes

@orcunc orcunc requested a review from kwart December 15, 2023 06:20
@orcunc
Copy link
Contributor Author

orcunc commented Dec 15, 2023

@kwart I have reverted .gitattributes file

Copy link
Contributor

@kwart kwart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One more legacy config caught my eye in POM files.

pom.xml Outdated Show resolved Hide resolved
smoke-test/pom.xml Outdated Show resolved Hide resolved
orcunc and others added 2 commits December 16, 2023 11:40
Co-authored-by: Josef Cacek <[email protected]>
Co-authored-by: Josef Cacek <[email protected]>
@orcunc orcunc requested a review from kwart December 16, 2023 08:47
@orcunc
Copy link
Contributor Author

orcunc commented Dec 16, 2023

@kwart applied suggested maven compiler changes

<dependency>
<groupId>org.junit.platform</groupId>
<artifactId>junit-platform-commons</artifactId>
<artifactId>junit-jupiter</artifactId>

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@orcunc orcunc merged commit f91d2b7 into hazelcast:main Dec 18, 2023
2 checks passed
@orcunc orcunc deleted the fix/hz-3617-upgrade-to-java-11 branch December 18, 2023 13:49
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.

4 participants