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 2021/09/23 08:59:45 UTC

[GitHub] [fineract] awasum commented on a change in pull request #1882: FINERACT-982 - Completely ditch use of Drizzle JDBC Driver after all

awasum commented on a change in pull request #1882:
URL: https://github.com/apache/fineract/pull/1882#discussion_r714601554



##########
File path: fineract-provider/src/main/resources/sql/migrations/core_db/V119__add_template_table.sql
##########
@@ -31,9 +31,9 @@ CREATE TABLE `m_template` (
 DROP TABLE IF EXISTS `m_templatemappers`;
 CREATE TABLE `m_templatemappers` (
   `id` BIGINT NOT NULL AUTO_INCREMENT,
-  `mapperkey` varchar(255) COLLATE utf8mb4_unicode_ci DEFAULT NULL,
+  `mapperkey` varchar(255) COLLATE utf8mb4_general_ci DEFAULT NULL,

Review comment:
       What happens to old deployments when u update and previous migration file? Can we instead update in a newer migration script?




-- 
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@fineract.apache.org

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