Uploaded image for project: 'Red Hat Decision Manager'
  1. Red Hat Decision Manager
  2. RHDM-733

Marshaller in kie-server retains old classes on KieScanner update

    Details

    • Target Release:
    • Story Points:
      2
    • Sprint:
      2018 Week 39-41
    • QE Test Coverage:
      +

      Description

      kie-server Marshaller retains the old version classes so they are used on the REST call. It will result in "rule is not fired" in the below scenario:

      • Create KieConatiner in kie-server with 1.0-SNAPSHOT version and enable KieScanner
      • [Build&Deploy] in business-central
      • Access kie-server -> works correctly
      • mvn deploy via command line (e.g. mvn clean install deploy -DaltDeploymentRepository=guvnor-m2-repo::default::http://localhost:8080/business-central/maven2/)
      • Access kie-server -> rule is not fired because of different versions of the same Fact class

      Marshaller is disposed on KieContainerInstanceImpl.updateReleaseId() but not on KieScanner update.

      https://github.com/kiegroup/droolsjbpm-integration/blob/master/kie-server-parent/kie-server-services/kie-server-services-common/src/main/java/org/kie/server/services/impl/KieContainerInstanceImpl.java#L232-L246

      Doesn't the comment in the method explain this problem?
      ~~~
      // marshallers need to disposed in case the container was updated with different releaseId
      // proper solution is to attach listener directly to the KieScanner and dispose the marshallers,
      // but those listeners are not (yet) available, so this is a temporary hackish "solution"
      ~~~

      But KieScannerEventListener is now available since RHDM 7.0 and BRMS 6.4.4.

      https://issues.jboss.org/browse/DROOLS-1565

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

                People

                • Assignee:
                  swiderski.maciej Maciej Swiderski
                  Reporter:
                  tkobayashi Toshiya Kobayashi
                  Tester:
                  Karel Suta
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: