[GitHub] [incubator-shardingsphere] KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-575017307
 
 
   > Add a field? like `private boolean enabled;`
   
   Not a good idea, the NoneStrategy already means do nothing. If set a sub 
field `enabled`, Is it means a true switch when `enabled=true`? I think it will 
confuse users.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-575014564
 
 
   I find the problem is caused by springboot. 
   springboot parse yaml and transfer to propertySource. At this time, 
springboot would ignore some item.
   
   I'm not sure whether there are some configuration so that springboot do not 
ignore them.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere-doc] haetao opened a new pull request #434: Add company to user list

2020-01-15 Thread GitBox
haetao opened a new pull request #434: Add company to user list
URL: https://github.com/apache/incubator-shardingsphere-doc/pull/434
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] tuohai666 opened a new pull request #3997: carry pr#3863 from dev to 4.0.1

2020-01-15 Thread GitBox
tuohai666 opened a new pull request #3997: carry pr#3863 from dev to 4.0.1
URL: https://github.com/apache/incubator-shardingsphere/pull/3997
 
 
   Fixes #3861.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] tuohai666 opened a new pull request #3996: carry pr#3969 from dev to 4.0.1

2020-01-15 Thread GitBox
tuohai666 opened a new pull request #3996: carry pr#3969 from dev to 4.0.1
URL: https://github.com/apache/incubator-shardingsphere/pull/3996
 
 
   Fixes #3968.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] haetao commented on issue #3993: Start error with yml

2020-01-15 Thread GitBox
haetao commented on issue #3993: Start error with yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3993#issuecomment-575009767
 
 
   To be exact, ShardingSphere's compatibility with MySQL 8 is not good.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere-doc] haetao opened a new pull request #433: Correct orchestration configuration of java

2020-01-15 Thread GitBox
haetao opened a new pull request #433: Correct orchestration configuration of 
java
URL: https://github.com/apache/incubator-shardingsphere-doc/pull/433
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] SteNicholas removed a comment on issue #3994: PostgreSQL current user

2020-01-15 Thread GitBox
SteNicholas removed a comment on issue #3994: PostgreSQL current user
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3994#issuecomment-574999262
 
 
   /ci


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] beijing-penguin commented on a change in pull request #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
beijing-penguin commented on a change in pull request #3988: Using 
multithreading make loadDefaultTables faster and remove repeat load 
tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#discussion_r367245631
 
 

 ##
 File path: 
sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java
 ##
 @@ -206,19 +207,46 @@ public TableMetas loadAll(final ShardingRule 
shardingRule) throws SQLException {
 return result;
 }
 
-private Map loadDefaultTables(final ShardingRule 
shardingRule) throws SQLException {
+private Map loadDefaultTables(final ShardingRule 
shardingRule, final Map result) throws SQLException {
 Optional actualDefaultDataSourceName = 
shardingRule.findActualDefaultDataSourceName();
 if (!actualDefaultDataSourceName.isPresent()) {
 return Collections.emptyMap();
 }
 Collection tableNames = 
loadAllTableNames(actualDefaultDataSourceName.get());
-Map result = new HashMap<>(tableNames.size(), 
1);
-for (String each : tableNames) {
-result.put(each, load(each, shardingRule));
+removeRepeatTable(tableNames, result);
+List metaList = 
executorEngine.execute(getTableNamesInput(tableNames), new 
GroupedCallback() {
+@Override
+public Collection execute(final Collection 
inputs, final boolean isTrunkThread, final Map dataMap) throws 
SQLException {
+String logicTableName = inputs.iterator().next();
+Collection result = new LinkedList<>();
+result.add(load(logicTableName, shardingRule));
+return result;
+}
+});
+Object[] tableNameArr = tableNames.toArray();
+for (int i = 0, size = tableNames.size(); i < size; i++) {
 
 Review comment:
   > Why not `for (int i = 0; i < tableNames.size(); i++) `?
   
   Avoid size() method multiple computations,,,for performance


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] beijing-penguin commented on a change in pull request #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
beijing-penguin commented on a change in pull request #3988: Using 
multithreading make loadDefaultTables faster and remove repeat load 
tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#discussion_r367245267
 
 

 ##
 File path: 
sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java
 ##
 @@ -206,19 +207,46 @@ public TableMetas loadAll(final ShardingRule 
shardingRule) throws SQLException {
 return result;
 }
 
-private Map loadDefaultTables(final ShardingRule 
shardingRule) throws SQLException {
+private Map loadDefaultTables(final ShardingRule 
shardingRule, final Map result) throws SQLException {
 Optional actualDefaultDataSourceName = 
shardingRule.findActualDefaultDataSourceName();
 if (!actualDefaultDataSourceName.isPresent()) {
 return Collections.emptyMap();
 }
 Collection tableNames = 
loadAllTableNames(actualDefaultDataSourceName.get());
-Map result = new HashMap<>(tableNames.size(), 
1);
-for (String each : tableNames) {
-result.put(each, load(each, shardingRule));
+removeRepeatTable(tableNames, result);
 
 Review comment:
   > Why did you remove repeat table? For performance or anything else?
For performance.
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] SteNicholas commented on issue #3994: PostgreSQL current user

2020-01-15 Thread GitBox
SteNicholas commented on issue #3994: PostgreSQL current user
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3994#issuecomment-574999262
 
 
   /ci


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] beijing-penguin commented on a change in pull request #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
beijing-penguin commented on a change in pull request #3988: Using 
multithreading make loadDefaultTables faster and remove repeat load 
tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#discussion_r367245008
 
 

 ##
 File path: 
sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java
 ##
 @@ -206,19 +207,46 @@ public TableMetas loadAll(final ShardingRule 
shardingRule) throws SQLException {
 return result;
 }
 
-private Map loadDefaultTables(final ShardingRule 
shardingRule) throws SQLException {
+private Map loadDefaultTables(final ShardingRule 
shardingRule, final Map result) throws SQLException {
 Optional actualDefaultDataSourceName = 
shardingRule.findActualDefaultDataSourceName();
 if (!actualDefaultDataSourceName.isPresent()) {
 return Collections.emptyMap();
 }
 Collection tableNames = 
loadAllTableNames(actualDefaultDataSourceName.get());
-Map result = new HashMap<>(tableNames.size(), 
1);
-for (String each : tableNames) {
-result.put(each, load(each, shardingRule));
+removeRepeatTable(tableNames, result);
 
 Review comment:
   > If there are two tables with the same name `t1` and different structures 
in ds0 and default ds seperately, do you think removeRepeatTable() could return 
correct result list?
   
   yes, load() methed  will be get out the same as tableMetaData by table 
name...loadShardingTables or loadDefaultTables execute load() method and only 
first parameter table name is different.
   
![image](https://user-images.githubusercontent.com/10703753/72498045-26afd300-3869-11ea-9d7e-4c6135c49a9d.png)
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] KomachiSion merged pull request #3991: Support scaling PostgreSQL

2020-01-15 Thread GitBox
KomachiSion merged pull request #3991: Support scaling PostgreSQL
URL: https://github.com/apache/incubator-shardingsphere/pull/3991
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] tuohai666 commented on issue #3994: PostgreSQL current user

2020-01-15 Thread GitBox
tuohai666 commented on issue #3994: PostgreSQL current user
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3994#issuecomment-574990441
 
 
   /ci


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] cielswift commented on issue #3993: Start error with yml

2020-01-15 Thread GitBox
cielswift commented on issue #3993: Start error with yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3993#issuecomment-574990293
 
 
   4.0 version does not support mysql8?  


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere-doc] haetao closed pull request #432: update community/content/poweredby doc

2020-01-15 Thread GitBox
haetao closed pull request #432: update community/content/poweredby doc
URL: https://github.com/apache/incubator-shardingsphere-doc/pull/432
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere-doc] haetao opened a new pull request #432: update community/content/poweredby doc

2020-01-15 Thread GitBox
haetao opened a new pull request #432: update community/content/poweredby doc
URL: https://github.com/apache/incubator-shardingsphere-doc/pull/432
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] KomachiSion commented on issue #3991: Support scaling PostgreSQL

2020-01-15 Thread GitBox
KomachiSion commented on issue #3991: Support scaling PostgreSQL
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3991#issuecomment-574982957
 
 
   /run ci


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] haetao commented on issue #3993: Start error with yml

2020-01-15 Thread GitBox
haetao commented on issue #3993: Start error with yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3993#issuecomment-574982247
 
 
   Hi! ShardingSphere supports mysql5 at this  version! Your driver is mysql8.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] geomonlin opened a new issue #3995: logicTable of ShardingSphereMetaData don't contain encrypt field in sharding-proxy when testing 'sharding+encrypt'

2020-01-15 Thread GitBox
geomonlin opened a new issue #3995: logicTable of ShardingSphereMetaData don't 
contain encrypt field in sharding-proxy when testing 'sharding+encrypt'
URL: https://github.com/apache/incubator-shardingsphere/issues/3995
 
 
   ## Bug Report
   
   **For English only**, other languages will not accept.
   
   Before report a bug, make sure you have:
   
   - Searched open and closed [GitHub 
issues](https://github.com/apache/incubator-shardingsphere/issues).
   - Read documentation: [ShardingSphere 
Doc](https://shardingsphere.apache.org/document/current/en/overview).
   
   Please pay attention on issues you submitted, because we maybe need more 
details. 
   If no response **more than 7 days** and we cannot reproduce it on current 
information, we will **close it**.
   
   Please answer these questions before submitting your issue. Thanks!
   
   ### Which version of ShardingSphere did you use?
   
   **dev(5.0)**
   
   ### Which project did you use? Sharding-JDBC or Sharding-Proxy?
   
   Sharding-Proxy
   
   ### Expected behavior
   tableMetaData contain encrypt field.
   
   ### Actual behavior
   tableMetaData don't contain encrypt field in MetaData.
   
   ### Reason analyze (If you can)
   
![image](https://user-images.githubusercontent.com/26007894/72493635-99ff1800-385c-11ea-9161-fc90b4586a16.png)
   
   
![image](https://user-images.githubusercontent.com/26007894/72493815-fcf0af00-385c-11ea-80cf-8c410ac04084.png)
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] SteNicholas edited a comment on issue #3990: Parse engine rejects CURRENT_USER in 4.0.0

2020-01-15 Thread GitBox
SteNicholas edited a comment on issue #3990: Parse engine rejects CURRENT_USER 
in 4.0.0
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3990#issuecomment-574947143
 
 
   @encodeering 4.0.0 PostgreSQLParser doesn't support `SELECT CURRENT_USER`, I 
would like to add the keyword to `simpleExpr`.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] SteNicholas commented on a change in pull request #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
SteNicholas commented on a change in pull request #3988: Using multithreading 
make loadDefaultTables faster and remove repeat load tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#discussion_r367220680
 
 

 ##
 File path: 
sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java
 ##
 @@ -206,19 +207,46 @@ public TableMetas loadAll(final ShardingRule 
shardingRule) throws SQLException {
 return result;
 }
 
-private Map loadDefaultTables(final ShardingRule 
shardingRule) throws SQLException {
+private Map loadDefaultTables(final ShardingRule 
shardingRule, final Map result) throws SQLException {
 Optional actualDefaultDataSourceName = 
shardingRule.findActualDefaultDataSourceName();
 if (!actualDefaultDataSourceName.isPresent()) {
 return Collections.emptyMap();
 }
 Collection tableNames = 
loadAllTableNames(actualDefaultDataSourceName.get());
-Map result = new HashMap<>(tableNames.size(), 
1);
-for (String each : tableNames) {
-result.put(each, load(each, shardingRule));
+removeRepeatTable(tableNames, result);
 
 Review comment:
   Why did you remove repeat table? For performance or anything else?


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] SteNicholas commented on a change in pull request #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
SteNicholas commented on a change in pull request #3988: Using multithreading 
make loadDefaultTables faster and remove repeat load tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#discussion_r367220920
 
 

 ##
 File path: 
sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java
 ##
 @@ -206,19 +207,46 @@ public TableMetas loadAll(final ShardingRule 
shardingRule) throws SQLException {
 return result;
 }
 
-private Map loadDefaultTables(final ShardingRule 
shardingRule) throws SQLException {
+private Map loadDefaultTables(final ShardingRule 
shardingRule, final Map result) throws SQLException {
 Optional actualDefaultDataSourceName = 
shardingRule.findActualDefaultDataSourceName();
 if (!actualDefaultDataSourceName.isPresent()) {
 return Collections.emptyMap();
 }
 Collection tableNames = 
loadAllTableNames(actualDefaultDataSourceName.get());
-Map result = new HashMap<>(tableNames.size(), 
1);
-for (String each : tableNames) {
-result.put(each, load(each, shardingRule));
+removeRepeatTable(tableNames, result);
+List metaList = 
executorEngine.execute(getTableNamesInput(tableNames), new 
GroupedCallback() {
+@Override
+public Collection execute(final Collection 
inputs, final boolean isTrunkThread, final Map dataMap) throws 
SQLException {
+String logicTableName = inputs.iterator().next();
+Collection result = new LinkedList<>();
+result.add(load(logicTableName, shardingRule));
+return result;
+}
+});
+Object[] tableNameArr = tableNames.toArray();
+for (int i = 0, size = tableNames.size(); i < size; i++) {
 
 Review comment:
   Why not `for (int i = 0; i < tableNames.size(); i++) `?


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] SteNicholas opened a new pull request #3994: PostgreSQL current user

2020-01-15 Thread GitBox
SteNicholas opened a new pull request #3994: PostgreSQL current user
URL: https://github.com/apache/incubator-shardingsphere/pull/3994
 
 
   Fixes #3990.
   
   PostgreSQLParser doesn't support `SELECT CURRENT_USER`, because `simpleExpr` 
of select statment lack of `CURRENT_USER` keyword.
   
   Changes proposed in this pull request:
   - add `CURRENT_USER` to `simpleExpr`, not `functionCall`.
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] leego86 commented on issue #3992: Sharding-JDBC lost insertValuesContexts when insert

2020-01-15 Thread GitBox
leego86 commented on issue #3992: Sharding-JDBC lost insertValuesContexts when 
insert
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3992#issuecomment-574964370
 
 
   sql parse exception 
   
![image](https://user-images.githubusercontent.com/31645800/72490619-b433f880-3852-11ea-94bc-0861fb4a41c1.png)
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] avalon566 commented on issue #3991: Support scaling PostgreSQL

2020-01-15 Thread GitBox
avalon566 commented on issue #3991: Support scaling PostgreSQL
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3991#issuecomment-574961967
 
 
   /run ci


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] cielswift opened a new issue #3993: Start error with yml

2020-01-15 Thread GitBox
cielswift opened a new issue #3993: Start error with yml
URL: https://github.com/apache/incubator-shardingsphere/issues/3993
 
 
   spring boot version 2.2
   jdk 13
   sharding 4.0.0-RC3
   
   I searched most of the cases and configured them with properties
   But I used yml, but something went wrong
   
   Failed to configure a DataSource: 'url' attribute is not specified and no 
embedded datasource could be configured.
   Reason: Failed to determine a suitable driver class
   
   This is my configuration
   
 shardingsphere:  
   datasource:
 names: ds0,ds1
 ds0:
   type: com.alibaba.druid.pool.DruidDataSource
   driverClassName: com.mysql.cj.jdbc.Driver
   url: 
jdbc:mysql://mysql.ciel.cl:1600/ds0?seUnicode=true=UTF-8=false=Asia/Shanghai
   username: root
   password: ciel
 ds1:
   type: com.alibaba.druid.pool.DruidDataSource
   driverClassName: com.mysql.cj.jdbc.Driver
   url: 
jdbc:mysql://mysql.ciel.cl:1601/ds1?seUnicode=true=UTF-8=false=Asia/Shanghai
   username: root
   password: ciel
   sharding: 
 default-database-strategy:
   inline:
 sharding-column: id
 algorithm-expression: ds$->{id % 2}
   
 tables:   
   t_order:
 actual-data-nodes: ds$->{0..1}.t_order$->{0..2}
 table-strategy:
   inline:
 sharding-column: id  
 algorithm-expression: t_order$->{id % 2}
 key-generator: # 
   column: id
   type: SNOWFLAKE
   #t_order_item:
   #  actual-data-nodes: ds$->{0..1}.t_order_item$->{0..1}
   #  table-strategy:
   #inline:
   #  sharding-column: order_id
   #  algorithm-expression: t_order_item$->{order_id % 2}
   props:
 sql:
   show: true #
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] leego86 opened a new issue #3992: Sharding-JDBC lost insertValuesContexts when insert

2020-01-15 Thread GitBox
leego86 opened a new issue #3992: Sharding-JDBC lost insertValuesContexts when 
insert
URL: https://github.com/apache/incubator-shardingsphere/issues/3992
 
 
   ## Bug Report
   
   **For English only**, other languages will not accept.
   
   Before report a bug, make sure you have:
   
   - Searched open and closed [GitHub 
issues](https://github.com/apache/incubator-shardingsphere/issues).
   - Read documentation: [ShardingSphere 
Doc](https://shardingsphere.apache.org/document/current/en/overview).
   
   Please pay attention on issues you submitted, because we maybe need more 
details. 
   If no response **more than 7 days** and we cannot reproduce it on current 
information, we will **close it**.
   
   Please answer these questions before submitting your issue. Thanks!
   
   ### Which version of ShardingSphere did you use?
4.0.0
   
   ### Which project did you use? Sharding-JDBC or Sharding-Proxy?
   Sharding-JDBC
   
   ### Expected behavior
   insert table success
   
   ### Actual behavior
   sql not use route, products all table sql and mybatis exception
   
   ### Reason analyze (If you can)
   I guess columns is too many(89) couse sql analysis exception when 
structuring sqlStatementContext,shardingConditions is empty because there is no 
insertValueContexts in sqlStatementContext,so not use table rule and setting 
parameters exception
   
  debug:
   
![image](https://user-images.githubusercontent.com/31645800/72487070-c3617900-3847-11ea-8b3c-500c9cf86f11.png)
   
   exception:
   
   ```
   org.mybatis.spring.MyBatisSystemException: nested exception is 
org.apache.ibatis.exceptions.PersistenceException: 
   ### Error updating database.  Cause: java.lang.IllegalStateException
   ### The error may involve defaultParameterMap
   ### The error occurred while setting parameters
   ### SQL: INSERT INTO orders ( id, orderId, outOrderId, orderType, state, 
createDate, paymentDate, issueDate, orderpayment, paymentStatus, 
splitTag, orderCategory, mergeDate, isSysMerge, mergeNumber, liangpinTag,   
  logisticStatus, logisticVendorType, logisticVendor, paidAmount, isUrgent, 
username, baseStore, paymentType, shad_phoneNumber, shad_mobilephone, 
shad_countrySubentity, shad_cityName, shad_name, shad_addressLine1, 
shad_addressLine2, shippingFirstName, shad_postalZone, shad_latitudeValue,  
   shad_longitudeValue, shad_countryIso3166Alpha2Code, shad_countryName, 
discount, carriageFee, serviceCharge, providerCode, version, source, 
buyerMemo, sellerMemo, invoicecontact_channelCode, 
invoicecontact_electronicMail, invoicecontact_name, invoicecontact_note, 
invoicecontact_telefax, invoicecontact_telephone, invoicecontact_value, 
invoicecontact_buyer, invoicecontact_taxpayernum, needinvoice, 
createdstate, prioritylevelcode, shippingtaxcategory, financeMemo, 
modifiedtime, reissuereason, storememo, scheduledshippingdate, 
oriorderid, lastname, reissuetype, contact_channelCode, 
contact_electronicMail, contact_name, contact_note, contact_telefax, 
contact_telephone, contact_value, parentorder, invoicetype, emailid, 
shippingandhandling, invoicecontent, shippinglastname, customerlocale, 
currencycode, employeeid, shippingmethod, firstname, creationtime, 
invoicetitle, expressSend, typecode, tenant, alipay_no ) VALUES 
( ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,   
  ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,  
   ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,
 1, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,  
   ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,
 ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?,   
  ?, ?, ?, 'OrderData', 'single', ? )  ,  ( ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, 1, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 'OrderData', 
'single', ? )  ,  ( ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, 1, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 
?, ?, ?, ?, ?, ?, ?, ?, 'OrderData', 'single', ? )  ,   
   ( 

[incubator-shardingsphere-benchmark] branch master updated: for performance

2020-01-15 Thread zhaoyanan
This is an automated email from the ASF dual-hosted git repository.

zhaoyanan pushed a commit to branch master
in repository 
https://gitbox.apache.org/repos/asf/incubator-shardingsphere-benchmark.git


The following commit(s) were added to refs/heads/master by this push:
 new dd73fe8  for performance
dd73fe8 is described below

commit dd73fe864d5ca1e8bb8da2ac6c8349da097094c7
Author: nancyzrh 
AuthorDate: Thu Jan 16 10:22:30 2020 +0800

for performance
---
 report/jtl_json/full_route_test.json| 2 +-
 report/jtl_json/master_slave_encrypt_sharding_test.json | 2 +-
 report/jtl_json/master_slave_test.json  | 2 +-
 report/jtl_json/single_route_test.json  | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/report/jtl_json/full_route_test.json 
b/report/jtl_json/full_route_test.json
index c3351f4..c936e95 100644
--- a/report/jtl_json/full_route_test.json
+++ b/report/jtl_json/full_route_test.json
@@ -1 +1 @@
-{"SELECT": [{"data": [{"99.9th": "9", "Err": "0.0%", "95th": "2", "50th": "1", 
"Max": "10", "Min": "0", "Throughout": "23036.87", "Samples": 4999, "Date": 
"2019.12.19 02:35:57 ", "Avg": "0"}, {"99.9th": "9", "Err": "0.0%", "95th": 
"2", "50th": "1", "Max": "9", "Min": "0", "Throughout": "22619.91", "Samples": 
4999, "Date": "2019.12.20 13:34:47 ", "Avg": "0"}, {"99.9th": "9", "Err": 
"0.0%", "95th": "2", "50th": "1", "Max": "9", "Min": "0", "Throughout": 
"22931.19", "Samples": 4999, "Date": [...]
\ No newline at end of file
+{"SELECT": [{"data": [{"99.9th": "9", "Err": "0.0%", "95th": "2", "50th": "1", 
"Max": "10", "Min": "0", "Throughout": "23036.87", "Samples": 4999, "Date": 
"2019.12.19 02:35:57 ", "Avg": "0"}, {"99.9th": "9", "Err": "0.0%", "95th": 
"2", "50th": "1", "Max": "9", "Min": "0", "Throughout": "22619.91", "Samples": 
4999, "Date": "2019.12.20 13:34:47 ", "Avg": "0"}, {"99.9th": "9", "Err": 
"0.0%", "95th": "2", "50th": "1", "Max": "9", "Min": "0", "Throughout": 
"22931.19", "Samples": 4999, "Date": [...]
\ No newline at end of file
diff --git a/report/jtl_json/master_slave_encrypt_sharding_test.json 
b/report/jtl_json/master_slave_encrypt_sharding_test.json
index 8ccfead..5d88667 100644
--- a/report/jtl_json/master_slave_encrypt_sharding_test.json
+++ b/report/jtl_json/master_slave_encrypt_sharding_test.json
@@ -1 +1 @@
-{"SELECT": [{"data": [{"99.9th": "16", "Err": "0.0%", "95th": "3", "50th": 
"2", "Max": "66", "Min": "1", "Throughout": "9381.73", "Samples": 99, 
"Date": "2019.12.19 02:40:33 ", "Avg": "2"}, {"99.9th": "16", "Err": "0.0%", 
"95th": "3", "50th": "2", "Max": "62", "Min": "1", "Throughout": "9038.48", 
"Samples": 99, "Date": "2019.12.20 13:39:23 ", "Avg": "2"}, {"99.9th": 
"16", "Err": "0.0%", "95th": "3", "50th": "2", "Max": "63", "Min": "1", 
"Throughout": "9071.44", "Samples": 99, [...]
\ No newline at end of file
+{"SELECT": [{"data": [{"99.9th": "16", "Err": "0.0%", "95th": "3", "50th": 
"2", "Max": "66", "Min": "1", "Throughout": "9381.73", "Samples": 99, 
"Date": "2019.12.19 02:40:33 ", "Avg": "2"}, {"99.9th": "16", "Err": "0.0%", 
"95th": "3", "50th": "2", "Max": "62", "Min": "1", "Throughout": "9038.48", 
"Samples": 99, "Date": "2019.12.20 13:39:23 ", "Avg": "2"}, {"99.9th": 
"16", "Err": "0.0%", "95th": "3", "50th": "2", "Max": "63", "Min": "1", 
"Throughout": "9071.44", "Samples": 99, [...]
\ No newline at end of file
diff --git a/report/jtl_json/master_slave_test.json 
b/report/jtl_json/master_slave_test.json
index 32238e1..47528fd 100644
--- a/report/jtl_json/master_slave_test.json
+++ b/report/jtl_json/master_slave_test.json
@@ -1 +1 @@
-{"INSERT+SELECT+DELETE": [{"data": [], "type": "Sharding-Proxy_3.1"}, {"data": 
[{"99.9th": "38", "Err": "0.0%", "95th": "16", "50th": "11", "Max": "48", 
"Min": "7", "Throughout": "1720.96", "Samples": 29709, "Date": "2020.01.01 
02:49:45 ", "Avg": "11"}, {"99.9th": "27", "Err": "0.0%", "95th": "16", "50th": 
"11", "Max": "37", "Min": "7", "Throughout": "1701.38", "Samples": 17218, 
"Date": "2020.01.02 02:49:52 ", "Avg": "11"}, {"99.9th": "30", "Err": "0.0%", 
"95th": "16", "50th": "11", "Max [...]
\ No newline at end of file
+{"INSERT+SELECT+DELETE": [{"data": [], "type": "Sharding-Proxy_3.1"}, {"data": 
[{"99.9th": "38", "Err": "0.0%", "95th": "16", "50th": "11", "Max": "48", 
"Min": "7", "Throughout": "1720.96", "Samples": 29709, "Date": "2020.01.01 
02:49:45 ", "Avg": "11"}, {"99.9th": "27", "Err": "0.0%", "95th": "16", "50th": 
"11", "Max": "37", "Min": "7", "Throughout": "1701.38", "Samples": 17218, 
"Date": "2020.01.02 02:49:52 ", "Avg": "11"}, {"99.9th": "30", "Err": "0.0%", 
"95th": "16", "50th": "11", "Max [...]
\ No newline at end of file
diff --git a/report/jtl_json/single_route_test.json 
b/report/jtl_json/single_route_test.json
index afff937..f1f4890 100644
--- a/report/jtl_json/single_route_test.json
+++ b/report/jtl_json/single_route_test.json
@@ -1 +1 @@
-{"SELECT": [{"data": [], "type": 

[GitHub] [incubator-shardingsphere] tmjGitAccount commented on issue #3964: Hintmanager databaseShardingOnly problem

2020-01-15 Thread GitBox
tmjGitAccount commented on issue #3964: Hintmanager  databaseShardingOnly 
problem
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3964#issuecomment-574947233
 
 
   @KomachiSion 
   
   spring:
 shardingsphere:
   props:
 sql:
   show: true
   sharding:
 default-database-strategy:
   hint:
 algorithm-class-name: com.a.b.c.OrderShardingAlgorithm
   datasource:
 names: dsa,dsb
 dsa:
   pool-name: dsa
   type: com.zaxxer.hikari.HikariDataSource
   driverClassName: com.mysql.jdbc.Driver
   jdbcUrl: xxx
   username: xxx
   password: xxx
 dsb:
   pool-name: dsb
   type: com.zaxxer.hikari.HikariDataSource
   driverClassName: com.mysql.jdbc.Driver
   jdbcUrl: xxx
   username: xxx
   password: xxx
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] SteNicholas commented on issue #3990: Parse engine rejects CURRENT_USER in 4.0.0

2020-01-15 Thread GitBox
SteNicholas commented on issue #3990: Parse engine rejects CURRENT_USER in 4.0.0
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3990#issuecomment-574947143
 
 
   @encodeering 4.0.0 doesn't support this function call, I will add to parse 
rules.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] 54czp commented on issue #2652: Sharding-Proxy4.0.0-RC1 Not available 。The download address is http://mirrors.tuna.tsinghua.edu.cn/apache/incubator/shardingsphere

2020-01-15 Thread GitBox
54czp commented on issue #2652: Sharding-Proxy4.0.0-RC1  Not available  。The 
download address is 
http://mirrors.tuna.tsinghua.edu.cn/apache/incubator/shardingsphere/4.0.0-RC1/apache-shardingsphere-incubating-4.0.0-RC1-sharding-proxy-bin.tar.gz
URL: 
https://github.com/apache/incubator-shardingsphere/issues/2652#issuecomment-574946478
 
 
   > download Sharding-Proxy4.0.0-RC1 ,and tar 。
   > According to the official website,
   > start bin/start.bat ,
   > it tells me 错误:找不到或无法加载主类 org.apache.shardingshpere.shardingproxy.Bootstrap
   > 
   > and i fond in Sharding-Proxy4.0.0-RC1 the jar was lost .
   > 
   > The download address is 
http://mirrors.tuna.tsinghua.edu.cn/apache/incubator/shardingsphere/4.0.0-RC1/apache-shardingsphere-incubating-4.0.0-RC1-sharding-proxy-bin.tar.gz
   
   Unzip-tar.gz.com is a free online service that allows you to convert 
any.tar.gz file into a.zip file that you can open with the default tools of 
your operating system.Just like you can access all the data stored in these 
files without installing special software.
   https://unzip-tar-gz.com/cn/
   
Unzip-tar.gz.com是一个免费的在线服务,它允许你任何.tar.gz文件转换成一个.zip文件,您可以与您的操作系统的默认工具打开。就像您可以访问存储在这些文件中的所有数据,而无需安装专用软件。
   
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] sunyinhui commented on issue #234: Welcome to leave your company's info(欢迎使用ShardingSphere的公司在此登记)

2020-01-15 Thread GitBox
sunyinhui commented on issue #234: Welcome to leave your company's 
info(欢迎使用ShardingSphere的公司在此登记)
URL: 
https://github.com/apache/incubator-shardingsphere/issues/234#issuecomment-574946223
 
 
   聚励云机械 https://www.cloudm.com


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] dongzl commented on issue #3973: 4.0.0 dosen't support LocalDateTime

2020-01-15 Thread GitBox
dongzl commented on issue #3973: 4.0.0 dosen't support LocalDateTime
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3973#issuecomment-574941239
 
 
   The 4.0.0 stable version was released just now, and 5.x is in development. 
   
   There is no exact release time yet. You can stay focused on ShardingSphere 
community.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] avalon566 opened a new pull request #3991: Support scaling PostgreSQL

2020-01-15 Thread GitBox
avalon566 opened a new pull request #3991: Support scaling PostgreSQL
URL: https://github.com/apache/incubator-shardingsphere/pull/3991
 
 
   For #3961 


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] encodeering opened a new issue #3990: Parse engine rejects CURRENT_USER in 4.0.0

2020-01-15 Thread GitBox
encodeering opened a new issue #3990: Parse engine rejects CURRENT_USER in 4.0.0
URL: https://github.com/apache/incubator-shardingsphere/issues/3990
 
 
   ## Bug Report
   
   ### Which version of ShardingSphere did you use?
   
   4.0.0
   
   ### Which project did you use? Sharding-JDBC or Sharding-Proxy?
   
   Sharding-JDBC
   
   ### Expected behavior
   
   Update from 3.1.0 to 4.0.0 would not change the parser rules.
   
   ### Actual behavior
   
   Integration tests fail because flyway can't perform a migration when 
executing `select CURRENT_USER`
   
   ### Reason analyze (If you can)
   
   ```
   org.postgresql:postgresql:42.2.6
   org.flywaydb:flyway-core:5.2.4
   org.testcontainers:postgresql:1.12.0
   
   sql = "SELECT CURRENT_USER"
   parseEngine = {SQLParseEngine@4966} 
databaseTypeName = "PostgreSQL"
   
   line 1:7 mismatched input 'CURRENT_USER' expecting {'U', 'u', '!', '~', '+', 
'-', '*', '(', '{', '?', INSERT, UPDATE, DELETE, ALTER, DROP, TRUNCATE, SCHEMA, 
REVOKE, ADD, SET, INDEX, KEY, FUNCTION, TRIGGER, PROCEDURE, DISTINCT, CASE, 
CAST, IF, NOT, NULL, TRUE, FALSE, EXISTS, ALL, BEGIN, COMMIT, ROLLBACK, 
SAVEPOINT, BOOLEAN, CHAR, INTERVAL, TIME, TIMESTAMP, LOCALTIME, LOCALTIMESTAMP, 
YEAR, MONTH, DAY, HOUR, MINUTE, SECOND, MAX, MIN, SUM, COUNT, AVG, CURRENT, 
ENABLE, DISABLE, INSTANCE, PRESERVE, DEFINER, SQL, CASCADED, LOCAL, CLOSE, 
OPEN, NEXT, NAME, NAMES, INTEGER, REAL, DECIMAL, TYPE, ADMIN, BINARY, ESCAPE, 
EXCLUDE, PARTITION, ROW, UNKNOWN, ALWAYS, CASCADE, GENERATED, ISOLATION, LEVEL, 
OPTION, PRIVILEGES, READ, ROLE, ROWS, START, TRANSACTION, ACTION, CACHE, 
CHARACTERISTICS, CLUSTER, COMMENTS, CONSTRAINTS, CURRENT_TIMESTAMP, CYCLE, 
DATA, DATABASE, DEFAULTS, DEFERRED, DEPENDS, DOMAIN, EXCLUDING, EXECUTE, 
EXTENDED, EXTENSION, EXTERNAL, EXTRACT, FILTER, FIRST, FOLLOWING, FORCE, 
GLOBAL, IDENTITY, IMMEDIATE, INCLUDING, INCREMENT, INDEXES, INHERIT, INHERITS, 
INCLUDE, LANGUAGE, LARGE, LAST, LOGGED, MAIN, MATCH, MAXVALUE, MINVALUE, 
NOTHING, NULLS, OBJECT, OIDS, OVER, OWNED, OWNER, PARTIAL, PLAIN, PRECEDING, 
RANGE, RENAME, REPLICA, RESET, RESTART, RESTRICT, ROUTINE, RULE, SECURITY, 
SEQUENCE, SESSION, SHOW, SIMPLE, STATISTICS, STORAGE, TABLESPACE, TEMP, 
TEMPORARY, UNBOUNDED, UNLOGGED, USAGE, VALID, VALIDATE, WITHIN, WITHOUT, ZONE, 
OF, GROUPS, RECURSIVE, IDENTIFIER_, STRING_, NUMBER_, HEX_DIGIT_, BIT_NUM_}
   
   java.lang.IllegalStateException
at 
com.google.common.base.Preconditions.checkState(Preconditions.java:429)
at 
org.apache.shardingsphere.sql.parser.core.extractor.util.ExtractorUtils.getFirstChildNode(ExtractorUtils.java:48)
at 
org.apache.shardingsphere.sql.parser.core.extractor.impl.dml.select.SelectItemsExtractor.extract(SelectItemsExtractor.java:54)
at 
org.apache.shardingsphere.sql.parser.core.extractor.SQLSegmentsExtractorEngine.extract(SQLSegmentsExtractorEngine.java:47)
at 
org.apache.shardingsphere.sql.parser.core.SQLParseKernel.parse(SQLParseKernel.java:59)
at 
org.apache.shardingsphere.sql.parser.SQLParseEngine.parse0(SQLParseEngine.java:70)
at 
org.apache.shardingsphere.sql.parser.SQLParseEngine.parse(SQLParseEngine.java:52)
at 
org.apache.shardingsphere.core.route.router.masterslave.MasterSlaveRouter.route(MasterSlaveRouter.java:56)
at 
org.apache.shardingsphere.shardingjdbc.jdbc.core.statement.MasterSlavePreparedStatement.(MasterSlavePreparedStatement.java:68)
at 
org.apache.shardingsphere.shardingjdbc.jdbc.core.statement.MasterSlavePreparedStatement.(MasterSlavePreparedStatement.java:53)
at 
org.apache.shardingsphere.shardingjdbc.jdbc.core.connection.MasterSlaveConnection.prepareStatement(MasterSlaveConnection.java:76)
at 
org.flywaydb.core.internal.jdbc.JdbcTemplate.prepareStatement(JdbcTemplate.java:333)
at 
org.flywaydb.core.internal.jdbc.JdbcTemplate.queryForString(JdbcTemplate.java:190)
at 
org.flywaydb.core.internal.database.postgresql.PostgreSQLConnection.(PostgreSQLConnection.java:48)
at 
org.flywaydb.core.internal.database.postgresql.PostgreSQLDatabase.getConnection(PostgreSQLDatabase.java:56)
at 
org.flywaydb.core.internal.database.postgresql.PostgreSQLDatabase.getConnection(PostgreSQLDatabase.java:31)
at 
org.flywaydb.core.internal.database.base.Database.getMainConnection(Database.java:366)
at org.flywaydb.core.Flyway.prepareSchemas(Flyway.java:1773)
at org.flywaydb.core.Flyway.execute(Flyway.java:1678)
at org.flywaydb.core.Flyway.migrate(Flyway.java:1356)
   ```
   
   ### Steps to reproduce the behavior, such as: SQL to execute, sharding rule 
configuration, when exception occur etc.
   
   Should be reproducible if the mentioned statement is executed with the 
PostgreSQL SQLParseEngine 
   
   ### Example codes for reproduce this issue (such as a github link).
   


This is an automated message from the Apache Git 

[GitHub] [incubator-shardingsphere] tristaZero merged pull request #3989: Add table owner and delimiter assertion

2020-01-15 Thread GitBox
tristaZero merged pull request #3989: Add table owner and delimiter assertion
URL: https://github.com/apache/incubator-shardingsphere/pull/3989
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] coveralls commented on issue #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
coveralls commented on issue #3988: Using multithreading make loadDefaultTables 
faster and remove repeat load tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#issuecomment-574639874
 
 
   ## Pull Request Test Coverage Report for [Build 
1381](https://coveralls.io/builds/28123225)
   
   * **0** of **20**   **(0.0%)**  changed or added relevant lines in **1** 
file are covered.
   * No unchanged relevant lines lost coverage.
   * Overall coverage decreased (**-0.06%**) to **65.893%**
   
   ---
   
   |  Changes Missing Coverage | Covered Lines | Changed/Added Lines | % |
   | :-|--||---: |
   | 
[sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java](https://coveralls.io/builds/28123225/source?filename=sharding-core%2Fsharding-core-execute%2Fsrc%2Fmain%2Fjava%2Forg%2Fapache%2Fshardingsphere%2Fsharding%2Fexecute%2Fmetadata%2Floader%2FShardingTableMetaDataLoader.java#L198)
 | 0 | 20 | 0.0%
   
   
   
   |  Totals | [![Coverage 
Status](https://coveralls.io/builds/28123225/badge)](https://coveralls.io/builds/28123225)
 |
   | :-- | --: |
   | Change from base [Build 689](https://coveralls.io/builds/28117346): |  
-0.06% |
   | Covered Lines: | 10815 |
   | Relevant Lines: | 16413 |
   
   ---
   #   - [Coveralls](https://coveralls.io)
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] devefx commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
devefx commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-574639640
 
 
   This my configuration
   
   ```yml
 shardingsphere:
   datasource:
 names: default, marketdata
 default:
   
 marketdata:
   xxx
   props:
 sql:
   show: true
   sharding:
 default-data-source-name: default
 default-database-strategy:
   standard:
 sharding-column: uid
 precise-algorithm-class-name: 
my.shardingsphere.strategy.DatabasePreciseShardingAlgorithm
 default-table-strategy:
   standard:
 sharding-column: uid
 precise-algorithm-class-name: 
my.shardingsphere.strategy.TablePreciseShardingAlgorithm
 tables:
   price_index:
 actual-data-nodes: marketdata.price_index
 database-strategy:
   none: {}
 table-strategy:
   none: {}
   
   ```


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] terrymanu opened a new pull request #3989: Add table owner and delimiter assertion

2020-01-15 Thread GitBox
terrymanu opened a new pull request #3989: Add table owner and delimiter 
assertion
URL: https://github.com/apache/incubator-shardingsphere/pull/3989
 
 
   For #3947.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] devefx commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
devefx commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-574638367
 
 
   
![image](https://user-images.githubusercontent.com/15070360/72433886-4b5a6b00-37d5-11ea-809f-d2b055a70875.png)
   
   ```java
   package org.apache.shardingsphere.core.yaml.config.sharding.strategy;
   
   import lombok.Data;
   
   @Data
   public final class YamlNoneShardingStrategyConfiguration implements 
YamlBaseShardingStrategyConfiguration {
   
   private boolean enabled;
   }
   ```
   This is me override
   
   
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-574638323
 
 
   Sorry, picture is not full configuration and quite inconvenient.
   Please paste congfiguration as a text to comment.
   You can remove some important data like username,password,url and so on.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] devefx commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
devefx commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-574636807
 
 
   
![image](https://user-images.githubusercontent.com/15070360/72433673-d71fc780-37d4-11ea-8ded-29d2b987ea71.png)
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-574635787
 
 
   For my test,
   
   ```yaml
   my_table:
 database-strategy:
   none: {}
   ```
   
   This configuration can work. 
   
   Can you provide your full sharding configuration? Or example code?
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] tristaZero commented on a change in pull request #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
tristaZero commented on a change in pull request #3988: Using multithreading 
make loadDefaultTables faster and remove repeat load tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#discussion_r366844620
 
 

 ##
 File path: 
sharding-core/sharding-core-execute/src/main/java/org/apache/shardingsphere/sharding/execute/metadata/loader/ShardingTableMetaDataLoader.java
 ##
 @@ -206,19 +207,46 @@ public TableMetas loadAll(final ShardingRule 
shardingRule) throws SQLException {
 return result;
 }
 
-private Map loadDefaultTables(final ShardingRule 
shardingRule) throws SQLException {
+private Map loadDefaultTables(final ShardingRule 
shardingRule, final Map result) throws SQLException {
 Optional actualDefaultDataSourceName = 
shardingRule.findActualDefaultDataSourceName();
 if (!actualDefaultDataSourceName.isPresent()) {
 return Collections.emptyMap();
 }
 Collection tableNames = 
loadAllTableNames(actualDefaultDataSourceName.get());
-Map result = new HashMap<>(tableNames.size(), 
1);
-for (String each : tableNames) {
-result.put(each, load(each, shardingRule));
+removeRepeatTable(tableNames, result);
 
 Review comment:
   If there are two tables with the same name `t1` and different structures in 
ds0 and default ds seperately, do you think removeRepeatTable() could return 
correct result list?


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] devefx commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
devefx commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-574633976
 
 
   > @devefx Can you try to use?
   > 
   > ```yaml
   > my_table:
   >   database-strategy:
   > none: {}
   > ```
   
   Still not working, a field must be included


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] tristaZero commented on issue #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
tristaZero commented on issue #3988: Using multithreading make 
loadDefaultTables faster and remove repeat load tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#issuecomment-574626422
 
 
   /ci


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] beijing-penguin commented on issue #3988: Using multithreading make loadDefaultTables faster and remove repeat load tableMetaData

2020-01-15 Thread GitBox
beijing-penguin commented on issue #3988: Using multithreading make 
loadDefaultTables faster and remove repeat load tableMetaData
URL: 
https://github.com/apache/incubator-shardingsphere/pull/3988#issuecomment-574592977
 
 
   /ci


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] beijing-penguin opened a new pull request #3988: Using multithreading make loadDefaultTables faster and remove repeat

2020-01-15 Thread GitBox
beijing-penguin opened a new pull request #3988: Using multithreading make 
loadDefaultTables faster and remove repeat
URL: https://github.com/apache/incubator-shardingsphere/pull/3988
 
 
   Fixes #3979.
   
   Changes proposed in this pull request:
   - Using multithreading make loadDefaultTables faster and remove repeat
   load tableMetaData


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] KomachiSion commented on issue #3964: Hintmanager databaseShardingOnly problem

2020-01-15 Thread GitBox
KomachiSion commented on issue #3964: Hintmanager  databaseShardingOnly problem
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3964#issuecomment-574584713
 
 
   @tmjGitAccount , Can you provide your full sharding configuration?


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] ette9844 closed issue #3987: How to handle JNDI datasource in spring namespace?

2020-01-15 Thread GitBox
ette9844 closed issue #3987: How to handle JNDI datasource in spring namespace?
URL: https://github.com/apache/incubator-shardingsphere/issues/3987
 
 
   


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, NoneShardingStrategy, does not work in yml

2020-01-15 Thread GitBox
KomachiSion commented on issue #3977: The sharding-JDBC non-sharding policy, 
NoneShardingStrategy, does not work in yml
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3977#issuecomment-574545912
 
 
   @devefx Can you try to use? 
   ```yaml
   my_table:
 database-strategy:
   none: {}
   ```


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [incubator-shardingsphere] joewee commented on issue #3856: Parse `ON DUPLICATE KEY UPDATE xxx= VALUES(xxx)` error

2020-01-15 Thread GitBox
joewee commented on issue #3856: Parse `ON DUPLICATE KEY UPDATE xxx= 
VALUES(xxx)` error
URL: 
https://github.com/apache/incubator-shardingsphere/issues/3856#issuecomment-574542715
 
 
   > > Hi, is there any update about this issue ? @SteNicholas @terrymanu
   > 
   > Sorry for the delay, and today I would validate the SQL.
   
   Thank you!


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:
us...@infra.apache.org


With regards,
Apache Git Services