feat(amazonq): java21 support #6414
Conversation
|
|
TO-DO: add changelog |
| @@ -483,6 +483,11 @@ export class GumbyController { | |||
| message.tabID | |||
| ) | |||
|
|
|||
| if (fromJDKVersion === JDKVersion.JDK21 && toJDKVersion === JDKVersion.JDK17) { | |||
There was a problem hiding this comment.
Why block this but allow other downgrades? e.g. 21 -> 11
There was a problem hiding this comment.
Only this one is possible
The source Java versions we show in dropdown form are: 8, 11, 17, 21
The target Java versions we show in dropdown form are: 17, 21
So the only downgrade possible for user to select is 21 -> 17
There was a problem hiding this comment.
So in other words you don't allow downgrades. This is a confusing if statement if that is the original intent, can we maybe check that source version is not >= target version? Or a comment at the very least.
There was a problem hiding this comment.
Added another comment to make it more specific but after that LGTM
| @@ -22,9 +22,12 @@ export const AWSTemplateKeyWords = ['AWSTemplateFormatVersion', 'Resources', 'AW | |||
|
|
|||
| export const AWSTemplateCaseInsensitiveKeyWords = ['cloudformation', 'cfn', 'template', 'description'] | |||
|
|
|||
| // TO-DO: make sure all Strings look good for Java 21 | |||
There was a problem hiding this comment.
Are you still working on this TODO or did we just want to keep it in?
There was a problem hiding this comment.
Strings should be good, will remove this comment when confirmed
| 'This diff patch covers the set of upgrades for Springboot, JUnit, and PowerMockito frameworks.', | ||
| 'This diff patch covers the set of upgrades for Springboot, JUnit, and PowerMockito frameworks in Java 17.', | ||
| 'Prepare minimal upgrade to Java 21': | ||
| 'This diff patch covers the set of upgrades for Springboot, JUnit, and PowerMockito frameworks in Java 21.', |
There was a problem hiding this comment.
I haven't looked at the closures for java 21 (I can look), but assuming they make the same corresponding upgrades as the first patch for java 17
There was a problem hiding this comment.
Yes will confirm, but that should be true
| @@ -436,6 +436,9 @@ export async function zipCode( | |||
| export async function startJob(uploadId: string) { | |||
| const sourceLanguageVersion = `JAVA_${transformByQState.getSourceJDKVersion()}` | |||
| const targetLanguageVersion = `JAVA_${transformByQState.getTargetJDKVersion()}` | |||
| // TO-DO: remove these logs statements | |||
| console.log('sourceLanguageVersion', sourceLanguageVersion) | |||
There was a problem hiding this comment.
Assuming you'll be removing these
There was a problem hiding this comment.
Oops I just saw the GitHub comments my bad!
|
Need to receive confirmation that backend is ready for Java 21, so converting this PR to draft so that it is not accidentally merged, but it is ready. |
Co-authored-by: Maxim Hayes <[email protected]>
Problem
Support transformations to Java 21.
Solution
Add support.
feature/xbranches will not be squash-merged at release time.