You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@fineract.apache.org by GitBox <gi...@apache.org> on 2020/04/17 22:16:34 UTC

[GitHub] [fineract] nemux commented on issue #757: escape all `grouping` in SQL files for MySQL 8 compatibility

nemux commented on issue #757: escape all `grouping` in SQL files for MySQL 8 compatibility
URL: https://github.com/apache/fineract/pull/757#issuecomment-615488050
 
 
   I started this, during my testing it seems to work fine (CRUD operations over entities) but i think it need to be tested deeply to be shure that nothing is break. One little observation, in when you specify the schema name in the query, you need to add the string ?serverTimezone=UTC this in in files: fineract-provider/src/main/java/org/apache/fineract/infrastructure/core/boot/db/DataSourceProperties.java line 51 
   
   ![imagen](https://user-images.githubusercontent.com/3644786/79618085-bf572880-80ce-11ea-853b-d961406530b4.png)
   
   and I created he file fineract-provider/src/main/resources/sql/migrations/list_db/V1_2__updated_connection_string_option.sql
   
   ![imagen](https://user-images.githubusercontent.com/3644786/79618142-e150ab00-80ce-11ea-8794-7fd33758ef2b.png)
   
   Originally in https://github.com/apache/fineract/pull/638
   

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services