apache / incubator-kie-kogito-runtimes

Kogito Runtimes - Kogito is a cloud-native business automation technology for building cloud-ready business applications.
http://kogito.kie.org
Apache License 2.0
544 stars 209 forks source link

ZonedDateTime in pojo used in kogito process error in creating proto files #1985

Closed debu999 closed 2 years ago

debu999 commented 2 years ago

Describe the bug

Hi team, We are using kogito for a while now and are not able to use LocalDateTime, LocalDate, ZonedDateTime as default data types in classes used in bpmn2 version. Getting error like jandex cant find short . We use mongo db persistence. What data types are supported for proto file generation in the platform. Please advise.

Expected behavior

We should have no errors for using datatypes like zonedatetime localdatetime etc. Now we only use String, Long and Date in our process variable classes. Please advise.

Actual behavior

error. Jandex does not have class for all datetime fields.

How to Reproduce?

take any example and use zonedatetime. The example should support platform independence.

Output of uname -a or ver

No response

Output of java -version

No response

GraalVM version (if different from Java)

No response

Kogito version or git rev (or at least Quarkus version if you are using Kogito via Quarkus platform BOM)

No response

Build tool (ie. output of mvnw --version or gradlew --version)

No response

Additional information

No response

cristianonicolai commented 2 years ago

@debu999 You can see the list of supported types based on the marshallers available here: https://github.com/kiegroup/kogito-runtimes/tree/main/jbpm/process-serialization-protobuf/src/main/java/org/kie/kogito/serialization/process/impl/marshallers As for temporal ones, only Date is supported, but I created the following JIRA to track this request: https://issues.redhat.com/browse/KOGITO-6756

cristianonicolai commented 2 years ago

@debu999 the fix should land in version 1.18.0, feel free to reopen this if the problem persists.