cristof commented on PR #142: URL: https://github.com/apache/openjpa/pull/142#issuecomment-4161967632
> > Hi, @rzo1 ! Great work! I've started to test the PR on different DBs. I got some errors (127) and failures (4) on a docker postgresql-18. I will rebuild everything using C locale to be on the safe side and test on MariaDB (LTS and latest), MySQL (LTS and latest). As soon as I have some results, I'll cycle back. > > Great - I expected failures (of the regular build) against various dbms. Do you run those tests with containers? If so - once you have some more results - I can run that as well locally. Yes, running in containers (official image of each db), dropping and recreating openjpa testing database in each run, building and testing openjpa also in container (temurin-21) with C locale. First issues found with posgresql seems to be a regression when using uuids types. The original impl of it uses UUID pg db type instead of commonly used varchar and the most recent version of your branch fails when trying to insert string instead of uuid (should work with UUID). With other dbs, it used varchar as does eclipselink with any db. I will have more time to probe and test it during Easter holiday. -- 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]
