flowable-engine icon indicating copy to clipboard operation
flowable-engine copied to clipboard

Corrected updateDeadLetterJob and updateSuspendedJob tables (#3450 #3454)

Open basclaessen opened this issue 3 years ago • 2 comments

Fixes #3450 #3454

Check List:

  • Unit tests: NO
  • Documentation: NO

basclaessen avatar Aug 29 '22 06:08 basclaessen

@tijsrademakers I think commit https://github.com/flowable/flowable-engine/commit/6ededd77098a95309ed6417cfd31066bacda9e78 broke some test cases. Can you take a look please?

org.flowable.standalone.escapeclause.HistoricTaskQueryEscapeClauseTest

testQueryByTaskNameLikeIgnoreCase

09:49:45,515 [main] ERROR org.flowable.common.engine.impl.interceptor.CommandContext  - Error while closing command context
org.apache.ibatis.exceptions.PersistenceException: 
### Error querying database.  Cause: org.h2.jdbc.JdbcSQLSyntaxErrorException: Syntax error in SQL statement "SELECT RES.*  \000a     \000a    from ACT_HI_TASKINST RES\000a     \000a     WHERE  lower(RES.NAME_) like lower(? [*]escape '|') \000a   \000a   \000a    order by RES.START_TIME_ asc"; expected "[, ::, AT, FORMAT, *, /, %, +, -, ||, NOT, IS, ILIKE, REGEXP, AND, OR, )"; SQL statement:
SELECT RES.*  
     
    from ACT_HI_TASKINST RES
     
     WHERE  lower(RES.NAME_) like lower(? escape '|') 
   
   
    order by RES.START_TIME_ asc [42001-214]

basclaessen avatar Aug 29 '22 08:08 basclaessen

You are right @basclaessen, we are already aware about it and are looking into it as we speak.

filiphr avatar Aug 29 '22 08:08 filiphr

Already merged with newer PR, thanks.

tijsrademakers avatar Oct 02 '22 10:10 tijsrademakers