Skip to content

DLPX-82803 [Backport of DLPX-81830] Upgrade Engine JDK to 8u345-b01#245

Merged
smeda-delphix merged 1 commit into6.0/stagefrom
dlpx/pr/smeda-delphix/0d88207d-d145-478b-8d5f-ede7fb4b7310
Aug 31, 2022
Merged

DLPX-82803 [Backport of DLPX-81830] Upgrade Engine JDK to 8u345-b01#245
smeda-delphix merged 1 commit into6.0/stagefrom
dlpx/pr/smeda-delphix/0d88207d-d145-478b-8d5f-ede7fb4b7310

Conversation

@smeda-delphix
Copy link
Copy Markdown
Contributor

@smeda-delphix smeda-delphix commented Aug 30, 2022

Upgrade engine JDK to latest version available for 6.0.17.0. A newer version of JDK was released to fix a compiler bug after changes were done in master. So, updating the latest version to master as well.

ab-pre-push - http://selfservice.jenkins.delphix.com/job/appliance-build-orchestrator-pre-push/3050/

@smeda-delphix smeda-delphix marked this pull request as ready for review August 30, 2022 19:55
Copy link
Copy Markdown
Contributor

@prakashsurya prakashsurya left a comment

Choose a reason for hiding this comment

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

Since you're making the two branches do the same thing, do we want to remove the conditional?

I'm OK leaving it, if we think we'll diverge again the next time we update the JDK.. and thus, we'd just add the conditional back.. but I wanted to ask..

@smeda-delphix
Copy link
Copy Markdown
Contributor Author

Yes, I would prefer leaving it for next time we update JDK.

@smeda-delphix smeda-delphix merged commit d5891f2 into 6.0/stage Aug 31, 2022
@smeda-delphix smeda-delphix deleted the dlpx/pr/smeda-delphix/0d88207d-d145-478b-8d5f-ede7fb4b7310 branch August 31, 2022 19:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants