elguardian commented on PR #2139: URL: https://github.com/apache/incubator-kie-kogito-apps/pull/2139#issuecomment-2470544745
@fjtirado @gmunozfe @yesamer @pefernan In this case the problem with H2 has surfaced the problem of column names. Sometimes we use names that are sql keywords for certain database. I would argue that even if we fix the H2 problem without altering the database, we should strive to a more proper naming revisiting columns and check if some future support database have it. I would revisit this to a more broader scope to check databases supported and decide from there (after analising the impact) what to do. but this will come back for sure at some point in the future. It is not urgent as there is workaround for this case, but it is not a proper solutions as we are going to need to revisit this at some point. I can only see two options when the time comes: 1. offer a proper orm file remapping the names of those conflicting columns (the thing here is that we need add that to the classpath 2. redefining column names. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
