terrymanu edited a comment on issue #2275: Reconstruct rewriting module URL: https://github.com/apache/incubator-shardingsphere/issues/2275#issuecomment-491602194 Some code review suggestion: - [x] XXXSQLRewriteEngine move to new package name as `engine` - [x] XXXSQLRewriteEngine need implement a interface name as `SQLRewriteEngine` - [x] MasterSlaveSQLRewriteEngine use @RequiredArgsConstructor - [x] Constructor don't need java-doc - [x] Don't use @SuppressWarnings("all") on class EncryptUpdateItemColumnPlaceholder - [x] EncryptUpdateItemColumnPlaceholder has lots of field can be null, consider about the design, maybe split assistedColumn and not has assistedColumn situation - [x] Question in EncryptWhereColumnPlaceholder, between should not supported - [ ] Should not call SQLUtil.getExactlyValue on rewrite module, current value should come from parse module. For example: IndexPlaceholder
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
