Re: [PR] kie-issues#2789: [sonataflow-operator] Implement DB Migrator Changes into SonataFlow Operator [incubator-kie-tools]

["wmedvede (via GitHub)" <gi...@apache.org>]

wmedvede commented on PR #2790:
URL: https://github.com/apache/incubator-kie-tools/pull/2790#issuecomment-2631746350

   Hi @rhkp ,
   while doing some testing I have found a small issue in the DB Migrator:
   
   Let me explain:
   
   A database produced by the DB Migrator looks like this:
   
   ![DataBase-DB-Migrator](https://github.com/user-attachments/assets/5618c25c-47f1-4584-8429-e57b72406d54)
   
   
   On the other hand, if we start the data index service (not using the db migrator) and we enable flyway for the data index, we can see a database like this:
   
   
   ![DataBase-DataIndex](https://github.com/user-attachments/assets/2a26bc6b-6868-4fb1-af6c-75dbefdb1733)
   
   So we have a 1.5.0 and 1.5.1 files that are included by the data index service but not by the db migrator.
   
   This is basically because the data index service has a dependency on the `persistence-commons` module that incorporates the missing files.
   
   Your work is good! but we need to make the db migrator incorporate the .sql files provided by that module.
   
   See here in the db migrator  build structure:
   
   ![MissingSqlFiles](https://github.com/user-attachments/assets/194479fc-7d3e-4a9a-8be5-9e9e618f59dd)
   
   would you mind take a look ?
   
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@kie.apache.org
For additional commands, e-mail: commits-help@kie.apache.org