Add Java 8 compatibility checks to CI

[tsmalls93]

Expected behavior

The SDK should work with Java 8 according to the Readme.

Actual behavior

I received the following error:

java.lang.UnsupportedClassVersionError: JVMCFRE199E bad major version 55.0 of class=jakarta/annotation/Resource, the maximum supported major version is 52.0; offset=6

Steps to reproduce the behavior

Run an app that uses javax.javaee-api 7.0 on Open Liberty that uses the SDK on version 6.6.0 or later.

[baywet]

HI @tsmalls93
Thanks for using the SDK and for reporting this.
This was caused by the upgrade to kiota dependencies 1.1.9 in #1970
Which in turns had upgraded jakarta annotations microsoft/kiota-java#1186
@Ndiritu I believe we should rollback that change in kiota libraries, and then cascade the updates. Thoughts?

[devillaccc]

I had the same issue, to resolve this I had upgraded my Java on the server to JDK11. good to have the java 8 support

[baywet]

Out of curiosity, why are you still running on Java 8 for server applications?

[tsmalls93]

Out of curiosity, why are you still running on Java 8 for server applications?

We use IBM WebSphere Liberty and you can only have one Java EE runtime at a time with that product. We have hundreds of applications, so it is not practical to update them all at once. We are working towards a Cloud Native architecture with Quarkus, but we are stuck with what we have until then.

[andrueastman]

HI @tsmalls93 Thanks for using the SDK and for reporting this. This was caused by the upgrade to kiota dependencies 1.1.9 in #1970 Which in turns had upgraded jakarta annotations microsoft/kiota-java#1186 @Ndiritu I believe we should rollback that change in kiota libraries, and then cascade the updates. Thoughts?

If this error can be caught at compile time, does it make sense to update the CI to probably run on Java 8 as well?

[Ndiritu]

HI @tsmalls93 Thanks for using the SDK and for reporting this. This was caused by the upgrade to kiota dependencies 1.1.9 in #1970 Which in turns had upgraded jakarta annotations microsoft/kiota-java#1186 @Ndiritu I believe we should rollback that change in kiota libraries, and then cascade the updates. Thoughts?

Makes sense to rollback for now.
Working on adding the compile time checks to the build workflows as suggested by @andrueastman

[Ndiritu]

@tsmalls93 this should be working from v6.9.0

[tsmalls93]

Thanks @Ndiritu, it is working now.

[Ndiritu]

Re-opening to track CI changes.