Opened 3 years ago

Closed 2 years ago

#19617 closed enhancement (implemented)

Metrics Java projects should all use java8

Reported by: iwakeh Owned by:
Priority: Medium Milestone:
Component: Metrics Version:
Severity: Normal Keywords:
Cc: Actual Points:
Parent ID: Points:
Reviewer: Sponsor:

Description

this is the parent issue for all the java projects.

Child Tickets

TicketTypeStatusOwnerSummary
#19618enhancementclosediwakehjava8 for Onionoo
#19620enhancementclosedall metrics-web java projects should use java 8
#19621enhancementcloseduse java 8 in CollecTor
#19622enhancementclosedmetrics-teamuse java 8 in DescripTor
#19623enhancementcloseduse java 8 for Exonerator

Change History (9)

comment:1 Changed 3 years ago by iwakeh

The decision was made in #18861.

DescripTor (aka metrics-lib) should be migrated last.

comment:2 Changed 2 years ago by iwakeh

Because of #19620 and to make it easier to start java 8 transitioning: Maybe metrics-base should also be set to Java 8 now?

Last edited 2 years ago by iwakeh (previous) (diff)

comment:3 Changed 2 years ago by karsten

Yes! But how do we make sure that metrics-lib gets migrated last? Can we override the setting there and wait for the next major metrics-lib release to upgrade to Java 8?

comment:4 in reply to:  3 Changed 2 years ago by iwakeh

Replying to karsten:

Yes! But how do we make sure that metrics-lib gets migrated last? Can we override the setting there and wait for the next major metrics-lib release to upgrade to Java 8?

I would define "migrated" as "having been released". Thus, metrics-base could be set to Java 8 now and the various products are migrated once their first release based on Java 8 is out. Upping metrics-base to Java 8 at first only affects development.

Actually, the order of releases doesn't really matter, just that the products depending on metrics-lib need to be based on Java 8 when they are supposed to use a metrics-lib release that is Java 8. Only to prevent hot-fix releases for older metrics-lib releases, metrics-lib should be released as last product using Java8 (just in case something comes up before everything uses metrics-lib based on Java 8).

comment:5 Changed 2 years ago by iwakeh

PS:

Overriding settings as you suggest in comment:3 only would be necessary, if metrics-lib would have to be released earlier than the other products.

comment:6 Changed 2 years ago by karsten

Makes sense! Should I make this change, or do you want to do that?

comment:7 Changed 2 years ago by iwakeh

Only one character in this branch.

comment:8 Changed 2 years ago by karsten

Merged!

comment:9 Changed 2 years ago by karsten

Resolution: implemented
Status: newclosed

All done. Closing.

Note: See TracTickets for help on using tickets.