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

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Done
    • Icon: Major Major
    • 7.2.0.GA
    • 7.0.1.GA
    • Kie-Server
    • 2
    • +
    • 2018 Week 39-41

      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

              swiderski.maciej Maciej Swiderski (Inactive)
              rhn-support-tkobayas Toshiya Kobayashi
              Karel Suta Karel Suta
              Karel Suta Karel Suta
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: