[GitHub] [shardingsphere] ThanoshanMV commented on a change in pull request #10253: Add SQL Definition for `ALTER SYSTEM` of Oracle Database

2021-05-10 Thread GitBox


ThanoshanMV commented on a change in pull request #10253:
URL: https://github.com/apache/shardingsphere/pull/10253#discussion_r629841182



##
File path: 
shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-oracle/src/main/antlr4/imports/oracle/DDLStatement.g4
##
@@ -1421,3 +1421,258 @@ leadCdbUriClause
 propertyClause
 : PROPERTY (SET | REMOVE) DEFAULT_CREDENTIAL EQ_ qualifiedCredentialName
 ;
+
+alterSystem
+: ALTER SYSTEM alterSystemOption
+;
+
+alterSystemOption
+: archiveLogClause
+| checkpointClause
+| checkDatafilesClause
+| distributedRecovClauses
+| flushClause
+| endSessionClauses
+| switchLogfileClause
+| suspendResumeClause
+| quiesceClauses
+| rollingMigrationClauses
+| rollingPatchClauses
+| alterSystemSecuriyClauses
+| affinityClauses
+| shutdownDispatcherClause
+| registerClause
+| setClause
+| resetClause
+| relocateClientClause
+| cancelSqlClause
+| flushPasswordfileMetadataCacheClause
+;
+
+archiveLogClause
+: ARCHIVE LOG instanceClause? (sequenceClause | changeClause | 
currentClause | groupClause | logfileClause | nextClause | allClause) 
toLocationClause?
+;
+
+checkpointClause
+: CHECKPOINT (GLOBAL | LOCAL)?
+;
+
+checkDatafilesClause
+: CHECK DATAFILES (GLOBAL | LOCAL)?
+;
+
+distributedRecovClauses
+: (ENABLE | DISABLE) DISTRIBUTED RECOVERY
+;
+
+flushClause
+: FLUSH flushClauseOption
+;
+
+endSessionClauses
+: (disconnectSessionClause | killSessionClause) (IMMEDIATE | NOREPLY)?
+;
+
+switchLogfileClause
+: SWITCH LOGFILE
+;
+
+suspendResumeClause
+: SUSPEND | RESUME
+;
+
+quiesceClauses
+: QUIESCE RESTRICTED | UNQUIESCE
+;
+
+rollingMigrationClauses
+: startRollingMigrationClause | stopRollingMigrationClause
+;
+
+rollingPatchClauses
+: startRollingPatchClause | stopRollingPatchClause
+;
+
+alterSystemSecuriyClauses
+: restrictedSessionClause | setEncryptionWalletOpenClause | 
setEncryptionWalletCloseClause | setEncryptionKeyClause
+;
+
+affinityClauses
+: enableAffinityClause | disableAffinityClause
+;
+
+shutdownDispatcherClause
+: SHUTDOWN IMMEDIATE? dispatcherName
+;
+
+registerClause
+: REGISTER
+;
+
+setClause
+: SET alterSystemSetClause+
+;
+
+resetClause
+: RESET alterSystemResetClause+
+;
+
+relocateClientClause
+: RELOCATE CLIENT clientId
+;
+
+cancelSqlClause
+: CANCEL SQL SQ_ sessionId serialNumber instanceId? sqlId? SQ_
+;
+
+flushPasswordfileMetadataCacheClause
+: FLUSH PASSWORDFILE_METADATA_CACHE
+;
+
+instanceClause
+: INSTANCE instanceName
+;
+
+sequenceClause
+: SEQUENCE numberLiterals

Review comment:
   I added `INTEGER_` before `NUMBER_` @wgy8283335 though the error comes 
like before:
   
![BuildFails](https://user-images.githubusercontent.com/48581379/117759246-14695d80-b241-11eb-8a66-70e3921f63e5.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




[GitHub] [shardingsphere] codecov-commenter edited a comment on pull request #10253: Add SQL Definition for `ALTER SYSTEM` of Oracle Database

2021-05-10 Thread GitBox


codecov-commenter edited a comment on pull request #10253:
URL: https://github.com/apache/shardingsphere/pull/10253#issuecomment-832413337


   # 
[Codecov](https://codecov.io/gh/apache/shardingsphere/pull/10253?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#10253](https://codecov.io/gh/apache/shardingsphere/pull/10253?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (d0d8858) into 
[master](https://codecov.io/gh/apache/shardingsphere/commit/064a8cb15a2204641210213ded80bdc0af268382?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (064a8cb) will **increase** coverage by `0.07%`.
   > The diff coverage is `50.00%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/shardingsphere/pull/10253/graphs/tree.svg?width=650=150=pr=ZvlXpWa7so_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/shardingsphere/pull/10253?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
   
   ```diff
   @@ Coverage Diff  @@
   ## master   #10253  +/-   ##
   
   + Coverage 68.22%   68.29%   +0.07% 
 Complexity  684  684  
   
 Files  1706 1708   +2 
 Lines 2919029194   +4 
 Branches   5235 5235  
   
   + Hits  1991419938  +24 
   + Misses 7734 7713  -21 
   - Partials   1542 1543   +1 
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/shardingsphere/pull/10253?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...sql/common/statement/ddl/AlterSystemStatement.java](https://codecov.io/gh/apache/shardingsphere/pull/10253/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLXN0YXRlbWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9zcWwvY29tbW9uL3N0YXRlbWVudC9kZGwvQWx0ZXJTeXN0ZW1TdGF0ZW1lbnQuamF2YQ==)
 | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
   | 
[...atement/oracle/ddl/OracleAlterSystemStatement.java](https://codecov.io/gh/apache/shardingsphere/pull/10253/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLXN0YXRlbWVudC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9zcWwvZGlhbGVjdC9zdGF0ZW1lbnQvb3JhY2xlL2RkbC9PcmFjbGVBbHRlclN5c3RlbVN0YXRlbWVudC5qYXZh)
 | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
   | 
[...r/statement/impl/OracleDDLStatementSQLVisitor.java](https://codecov.io/gh/apache/shardingsphere/pull/10253/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLWRpYWxlY3Qvc2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci1vcmFjbGUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NxbC9wYXJzZXIvb3JhY2xlL3Zpc2l0b3Ivc3RhdGVtZW50L2ltcGwvT3JhY2xlRERMU3RhdGVtZW50U1FMVmlzaXRvci5qYXZh)
 | `84.55% <100.00%> (+0.12%)` | `1.00 <0.00> (ø)` | |
   | 
[...sphere/sql/parser/core/visitor/SQLVisitorRule.java](https://codecov.io/gh/apache/shardingsphere/pull/10253/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtc3FsLXBhcnNlci9zaGFyZGluZ3NwaGVyZS1zcWwtcGFyc2VyLWVuZ2luZS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvc3FsL3BhcnNlci9jb3JlL3Zpc2l0b3IvU1FMVmlzaXRvclJ1bGUuamF2YQ==)
 | `100.00% <100.00%> (ø)` | `0.00 <0.00> (ø)` | |
   | 
[...ing/core/job/task/incremental/IncrementalTask.java](https://codecov.io/gh/apache/shardingsphere/pull/10253/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtc2NhbGluZy9zaGFyZGluZ3NwaGVyZS1zY2FsaW5nLWNvcmUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL3NjYWxpbmcvY29yZS9qb2IvdGFzay9pbmNyZW1lbnRhbC9JbmNyZW1lbnRhbFRhc2suamF2YQ==)
 | `75.00% <0.00%> (+2.08%)` | `0.00% <0.00%> (ø%)` | |
   | 

[GitHub] [shardingsphere] KolorYan edited a comment on issue #7998: "No value specified for parameter" On INSERT

2021-05-10 Thread GitBox


KolorYan edited a comment on issue #7998:
URL: https://github.com/apache/shardingsphere/issues/7998#issuecomment-837699596


   dose this issue fixed?
   I had the same problem for sql with password function :
   
   INSERT INTO xxx (create_time, modify_time, is_deleted, 
 supplier_name, route_id, status, account, password,
 contacts, phone, email, app_id, app_secret, channel_types, 
switch_status,
 def_channel_type, api_type, supplier_type, sync_type_id)
   VALUES (NOW(), NOW(), #{isDeleted,jdbcType=TINYINT},
 #{supplierName,jdbcType=VARCHAR}, #{routeId,jdbcType=BIGINT}, 
#{status},
 #{account}, **password**(#{password}), #{contacts}, #{phone},
 #{email}, #{appId}, #{appSecret}, #{channelTypes}, #{switchStatus},
 #{defChannelType}, #{apiType}, #{supplierType}, #{syncTypeId})
   


-- 
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




[GitHub] [shardingsphere] huanghao495430759 closed pull request #10295: Add GLOBAL_RULE_NODE in RegistryCenterNode,add global rule listener and event.

2021-05-10 Thread GitBox


huanghao495430759 closed pull request #10295:
URL: https://github.com/apache/shardingsphere/pull/10295


   


-- 
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




[GitHub] [shardingsphere] KolorYan commented on issue #7998: "No value specified for parameter" On INSERT

2021-05-10 Thread GitBox


KolorYan commented on issue #7998:
URL: https://github.com/apache/shardingsphere/issues/7998#issuecomment-837699596


   has this issue fixed?
   I had the same problem for sql with password function :
   
   INSERT INTO xxx (create_time, modify_time, is_deleted, 
 supplier_name, route_id, status, account, password,
 contacts, phone, email, app_id, app_secret, channel_types, 
switch_status,
 def_channel_type, api_type, supplier_type, sync_type_id)
   VALUES (NOW(), NOW(), #{isDeleted,jdbcType=TINYINT},
 #{supplierName,jdbcType=VARCHAR}, #{routeId,jdbcType=BIGINT}, 
#{status},
 #{account}, password(#{password}), #{contacts}, #{phone},
 #{email}, #{appId}, #{appSecret}, #{channelTypes}, #{switchStatus},
 #{defChannelType}, #{apiType}, #{supplierType}, #{syncTypeId})
   


-- 
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




[GitHub] [shardingsphere-elasticjob] chiangyn commented on issue #1889: IpUtils.getIp()方法获取不到ip

2021-05-10 Thread GitBox


chiangyn commented on issue #1889:
URL: 
https://github.com/apache/shardingsphere-elasticjob/issues/1889#issuecomment-837680257


   https://www.cnblogs.com/passedbylove/p/7561006.html 有人讨论过JDK bug这个问题


-- 
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




[GitHub] [shardingsphere-elasticjob] chiangyn commented on issue #1889: IpUtils.getIp()方法获取不到ip

2021-05-10 Thread GitBox


chiangyn commented on issue #1889:
URL: 
https://github.com/apache/shardingsphere-elasticjob/issues/1889#issuecomment-837676773


   这是InetAddress..isReachable java方法问题,在某些操作系统下会取不到ip。建议替换这个方法的使用


-- 
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




[GitHub] [shardingsphere] zuoyedalaohu opened a new issue #10301: sharding-jdbc(4.1.1)String index out of range: 0

2021-05-10 Thread GitBox


zuoyedalaohu opened a new issue #10301:
URL: https://github.com/apache/shardingsphere/issues/10301


   ### SQL: SELECT concat(tab.cust_type,tab.cust_no) as 
id,tab.cust_no,tab.ub_cust_no,tab.cust_name, 
tab.cust_type,tab.salesman_code,tab.salesman_name,tab.salesman_dept_no, 
tab.salesman_dept_name,ifnull(tab.link_telphone,'') 
link_telphone,ifnull(tab.link_person,'') link_person,tab.if_limit, 
tab.limit_no,IFNULL(lmt.available_limit_amt,0) 
available_limit_amt,tab.belong_company, 
tab.state,tab.open_bank,tab.account_no from ( SELECT
 ci.cust_no, ci.ub_cust_no, ci.cust_name,   
  'JZ' AS cust_type, ci.salesman_code, 
ci.salesman_name, ci.salesman_dept_no, 
ci.salesman_dept_name, ci.link_telphone, 
ci.link_person, ci.if_limit, ci.limit_no,   
  ci.belong_company, ci.mnemonic_code, 
ci.state, '' as open_bank,   
   '' as account_no FROM 
op_deco_comp_info ci WHERE 1 = 1
   and 
ci.belong_company in  (   ? 
 )  AND 
ci.state = ? UNION ALL  
   SELECT si.cust_no, si.ub_cust_no,
 si.cust_name, 'GY' AS cust_type, 
si.salesman_code as salesman_code, si.salesman_name as 
salesman_name, si.salesman_dept_no as salesman_dept_no, 
si.salesman_dept_name as salesman_dept_name, 
si.link_telphone, si.link_person, '' as 
if_limit, '' as limit_no, si.belong_company,

  si.mnemonic_code, si.state, '' as open_bank,  
   '' as account_no FROM 
op_supplier_info si WHERE 1 = 1 
  and 
si.belong_company in  (   ? 
 )  AND 
si.state = ? UNION ALL  
   SELECT ti.cust_no, ti.ub_cust_no,
 ti.cust_name, 'YS' AS cust_type, '' as 
salesman_code, '' as salesman_name, '' as 
salesman_dept_no, '' as salesman_dept_name, 
ti.link_telephone as link_telphone, '' as link_person,  
   '' as if_limit, '' as limit_no,   
   ti.belong_company, ti.mnemonic_code, 
ti.state, ti.open_bank as open_bank, 
ti.account_no as account_no FROM 
op_transporter_info ti WHERE 1 = 1  
 
and ti.belong_company in  (   ? 
 )  AND 
ti.state = ? UNION ALL  
   SELECT ci.custom_no as cust_no, 
cr.ub_cust_no, ci.custom_name as cust_name, 
'GR' AS cust_type, cr.salesman_code, 
cr.salesman_name, cr.salesman_dept_no, 
cr.salesman_dept_name, ci.telephone as link_telphone,   
  ci.custom_name  as link_pe
 rson, cr.if_limit, cr.limit_no,
 cr.belong_company, ci.mnemonic_code, cr.state, 
'' as open_bank, '' as account_no 
FROM oper_custom_info ci,oper_custom_right cr WHERE 
1 = 1 and ci.custom_no = cr.custom_no   
and 
cr.belong_company in  (   ? 
 )  AND 
cr.state = ? ) tab LEFT 
JOIN op_limit_info lmt ON tab.limit_no = lmt.limit_no where 1=1
   ### Cause: java.lang.StringIndexOutOfBoundsException: String 

[GitHub] [shardingsphere] zuoyedalaohu opened a new issue #10300: sharding-jdbc(4.1.1) UNION ALL limit Exception

2021-05-10 Thread GitBox


zuoyedalaohu opened a new issue #10300:
URL: https://github.com/apache/shardingsphere/issues/10300


   [c.x.c.o.c.d.m.c.extOper.OpCustomMapper.getCustList.debug:143] ==>  
Preparing: SELECT concat(tab.cust_type,tab.cust_no) as 
id,tab.cust_no,tab.ub_cust_no,tab.cust_name, 
tab.cust_type,tab.salesman_code,tab.salesman_name,tab.salesman_dept_no, 
tab.salesman_dept_name,ifnull(tab.link_telphone,'') 
link_telphone,ifnull(tab.link_person,'') link_person,tab.if_limit, 
tab.limit_no,IFNULL(lmt.available_limit_amt,0) 
available_limit_amt,tab.belong_company, tab.state,tab.open_bank,tab.account_no 
from ( SELECT ci.cust_no, ci.ub_cust_no, ci.cust_name, 'JZ' AS cust_type, 
ci.salesman_code, ci.salesman_name, ci.salesman_dept_no, ci.salesman_dept_name, 
ci.link_telphone, ci.link_person, ci.if_limit, ci.limit_no, ci.belong_company, 
ci.mnemonic_code, ci.state, null as open_bank, null as account_no FROM 
op_deco_comp_info ci WHERE 1 = 1 and ci.belong_company in ( ? ) AND ci.state = 
? UNION ALL SELECT si.cust_no, si.ub_cust_no, si.cust_name, 'GY' AS cust_type, 
si.salesman_code as salesman_code, si.salesman_n
 ame as salesman_name, si.salesman_dept_no as salesman_dept_no, 
si.salesman_dept_name as salesman_dept_name, si.link_telphone, si.link_person, 
'' as if_limit, '' as limit_no, si.belong_company, si.mnemonic_code, si.state, 
'' as open_bank, '' as account_no FROM op_supplier_info si WHERE 1 = 1 and 
si.belong_company in ( ? ) AND si.state = ? UNION ALL SELECT ti.cust_no, 
ti.ub_cust_no, ti.cust_name, 'YS' AS cust_type, '' as salesman_code, '' as 
salesman_name, '' as salesman_dept_no, '' as salesman_dept_name, 
ti.link_telephone as link_telphone, '' as link_person, '' as if_limit, '' as 
limit_no, ti.belong_company, ti.mnemonic_code, ti.state, ti.open_bank as 
open_bank, ti.account_no as account_no FROM op_transporter_info ti WHERE 1 = 1 
and ti.belong_company in ( ? ) AND ti.state = ? UNION ALL SELECT ci.custom_no 
as cust_no, cr.ub_cust_no, ci.custom_name as cust_name, 'GR' AS cust_type, 
cr.salesman_code, cr.salesman_name, cr.salesman_dept_no, cr.salesman_dept_name, 
ci.telephone as link_telph
 one, ci.custom_name as link_person, cr.if_limit, cr.limit_no, 
cr.belong_company, ci.mnemonic_code, cr.state, '' as open_bank, '' as 
account_no FROM oper_custom_info ci,oper_custom_right cr WHERE 1 = 1 and 
ci.custom_no = cr.custom_no and cr.belong_company in ( ? ) AND cr.state = ? ) 
tab LEFT JOIN op_limit_info lmt ON tab.limit_no = lmt.limit_no where 1=1 limit 
? , ? 
   [c.x.c.o.c.d.m.c.extOper.OpCustomMapper.getCustList.debug:143] ==> 
Parameters: D00193(String), 1(String), D00193(String), 1(String), 
D00193(String), 1(String), D00193(String), 1(String), 0(Integer), 20(Integer)
   
   
   Caused by: java.lang.ClassCastException: java.lang.String cannot be cast to 
java.lang.Integer
at 
org.apache.shardingsphere.sql.parser.binder.segment.select.pagination.PaginationContext.getValue(PaginationContext.java:57)
at 
org.apache.shardingsphere.sql.parser.binder.segment.select.pagination.PaginationContext.(PaginationContext.java:50)
at 
org.apache.shardingsphere.sql.parser.binder.segment.select.pagination.engine.LimitPaginationContextEngine.createPaginationContext(LimitPaginationContextEngine.java:38)
at 
org.apache.shardingsphere.sql.parser.binder.segment.select.pagination.engine.PaginationContextEngine.createPaginationContext(PaginationContextEngine.java:48)
at 
org.apache.shardingsphere.sql.parser.binder.statement.dml.SelectStatementContext.(SelectStatementContext.java:100)
at 
org.apache.shardingsphere.sql.parser.binder.SQLStatementContextFactory.getDMLStatementContext(SQLStatementContextFactory.java:103)
at 
org.apache.shardingsphere.sql.parser.binder.SQLStatementContextFactory.newInstance(SQLStatementContextFactory.java:87)
at 
org.apache.shardingsphere.underlying.route.DataNodeRouter.createRouteContext(DataNodeRouter.java:99)
at 
org.apache.shardingsphere.underlying.route.DataNodeRouter.executeRoute(DataNodeRouter.java:89)
at 
org.apache.shardingsphere.underlying.route.DataNodeRouter.route(DataNodeRouter.java:76)
at 
org.apache.shardingsphere.underlying.pluggble.prepare.PreparedQueryPrepareEngine.route(PreparedQueryPrepareEngine.java:54)
at 
org.apache.shardingsphere.underlying.pluggble.prepare.BasePrepareEngine.executeRoute(BasePrepareEngine.java:96)
at 
org.apache.shardingsphere.underlying.pluggble.prepare.BasePrepareEngine.prepare(BasePrepareEngine.java:83)
at 
org.apache.shardingsphere.shardingjdbc.jdbc.core.statement.ShardingPreparedStatement.prepare(ShardingPreparedStatement.java:183)
at 
org.apache.shardingsphere.shardingjdbc.jdbc.core.statement.ShardingPreparedStatement.execute(ShardingPreparedStatement.java:143)
at sun.reflect.GeneratedMethodAccessor207.invoke(Unknown Source)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)

[GitHub] [shardingsphere] zexho994 closed issue #10290: application run failed for 5.0.0-alpha

2021-05-10 Thread GitBox


zexho994 closed issue #10290:
URL: https://github.com/apache/shardingsphere/issues/10290


   


-- 
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




[GitHub] [shardingsphere] zexho994 commented on issue #10290: application run failed for 5.0.0-alpha

2021-05-10 Thread GitBox


zexho994 commented on issue #10290:
URL: 
https://github.com/apache/shardingsphere/issues/10290#issuecomment-837637635


   @fengying-wzm 非常感谢,已解决,是配置的问题


-- 
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




[GitHub] [shardingsphere] wgy8283335 commented on a change in pull request #10253: Add SQL Definition for `ALTER SYSTEM` of Oracle Database

2021-05-10 Thread GitBox


wgy8283335 commented on a change in pull request #10253:
URL: https://github.com/apache/shardingsphere/pull/10253#discussion_r629757456



##
File path: 
shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-oracle/src/main/antlr4/imports/oracle/DDLStatement.g4
##
@@ -1421,3 +1421,258 @@ leadCdbUriClause
 propertyClause
 : PROPERTY (SET | REMOVE) DEFAULT_CREDENTIAL EQ_ qualifiedCredentialName
 ;
+
+alterSystem
+: ALTER SYSTEM alterSystemOption
+;
+
+alterSystemOption
+: archiveLogClause
+| checkpointClause
+| checkDatafilesClause
+| distributedRecovClauses
+| flushClause
+| endSessionClauses
+| switchLogfileClause
+| suspendResumeClause
+| quiesceClauses
+| rollingMigrationClauses
+| rollingPatchClauses
+| alterSystemSecuriyClauses
+| affinityClauses
+| shutdownDispatcherClause
+| registerClause
+| setClause
+| resetClause
+| relocateClientClause
+| cancelSqlClause
+| flushPasswordfileMetadataCacheClause
+;
+
+archiveLogClause
+: ARCHIVE LOG instanceClause? (sequenceClause | changeClause | 
currentClause | groupClause | logfileClause | nextClause | allClause) 
toLocationClause?
+;
+
+checkpointClause
+: CHECKPOINT (GLOBAL | LOCAL)?
+;
+
+checkDatafilesClause
+: CHECK DATAFILES (GLOBAL | LOCAL)?
+;
+
+distributedRecovClauses
+: (ENABLE | DISABLE) DISTRIBUTED RECOVERY
+;
+
+flushClause
+: FLUSH flushClauseOption
+;
+
+endSessionClauses
+: (disconnectSessionClause | killSessionClause) (IMMEDIATE | NOREPLY)?
+;
+
+switchLogfileClause
+: SWITCH LOGFILE
+;
+
+suspendResumeClause
+: SUSPEND | RESUME
+;
+
+quiesceClauses
+: QUIESCE RESTRICTED | UNQUIESCE
+;
+
+rollingMigrationClauses
+: startRollingMigrationClause | stopRollingMigrationClause
+;
+
+rollingPatchClauses
+: startRollingPatchClause | stopRollingPatchClause
+;
+
+alterSystemSecuriyClauses
+: restrictedSessionClause | setEncryptionWalletOpenClause | 
setEncryptionWalletCloseClause | setEncryptionKeyClause
+;
+
+affinityClauses
+: enableAffinityClause | disableAffinityClause
+;
+
+shutdownDispatcherClause
+: SHUTDOWN IMMEDIATE? dispatcherName
+;
+
+registerClause
+: REGISTER
+;
+
+setClause
+: SET alterSystemSetClause+
+;
+
+resetClause
+: RESET alterSystemResetClause+
+;
+
+relocateClientClause
+: RELOCATE CLIENT clientId
+;
+
+cancelSqlClause
+: CANCEL SQL SQ_ sessionId serialNumber instanceId? sqlId? SQ_

Review comment:
   I think so. @ThanoshanMV 

##
File path: 
shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-oracle/src/main/antlr4/imports/oracle/DDLStatement.g4
##
@@ -1421,3 +1421,258 @@ leadCdbUriClause
 propertyClause
 : PROPERTY (SET | REMOVE) DEFAULT_CREDENTIAL EQ_ qualifiedCredentialName
 ;
+
+alterSystem
+: ALTER SYSTEM alterSystemOption
+;
+
+alterSystemOption
+: archiveLogClause
+| checkpointClause
+| checkDatafilesClause
+| distributedRecovClauses
+| flushClause
+| endSessionClauses
+| switchLogfileClause
+| suspendResumeClause
+| quiesceClauses
+| rollingMigrationClauses
+| rollingPatchClauses
+| alterSystemSecuriyClauses
+| affinityClauses
+| shutdownDispatcherClause
+| registerClause
+| setClause
+| resetClause
+| relocateClientClause
+| cancelSqlClause
+| flushPasswordfileMetadataCacheClause
+;
+
+archiveLogClause
+: ARCHIVE LOG instanceClause? (sequenceClause | changeClause | 
currentClause | groupClause | logfileClause | nextClause | allClause) 
toLocationClause?
+;
+
+checkpointClause
+: CHECKPOINT (GLOBAL | LOCAL)?
+;
+
+checkDatafilesClause
+: CHECK DATAFILES (GLOBAL | LOCAL)?
+;
+
+distributedRecovClauses
+: (ENABLE | DISABLE) DISTRIBUTED RECOVERY
+;
+
+flushClause
+: FLUSH flushClauseOption
+;
+
+endSessionClauses
+: (disconnectSessionClause | killSessionClause) (IMMEDIATE | NOREPLY)?
+;
+
+switchLogfileClause
+: SWITCH LOGFILE
+;
+
+suspendResumeClause
+: SUSPEND | RESUME
+;
+
+quiesceClauses
+: QUIESCE RESTRICTED | UNQUIESCE
+;
+
+rollingMigrationClauses
+: startRollingMigrationClause | stopRollingMigrationClause
+;
+
+rollingPatchClauses
+: startRollingPatchClause | stopRollingPatchClause
+;
+
+alterSystemSecuriyClauses
+: restrictedSessionClause | setEncryptionWalletOpenClause | 
setEncryptionWalletCloseClause | setEncryptionKeyClause
+;
+
+affinityClauses
+: enableAffinityClause | disableAffinityClause
+;
+
+shutdownDispatcherClause
+: SHUTDOWN IMMEDIATE? dispatcherName
+;
+
+registerClause
+: REGISTER
+;
+
+setClause
+: SET alterSystemSetClause+
+;
+
+resetClause
+: RESET alterSystemResetClause+
+;
+
+relocateClientClause
+: RELOCATE 

[GitHub] [shardingsphere] wgy8283335 commented on a change in pull request #10253: Add SQL Definition for `ALTER SYSTEM` of Oracle Database

2021-05-10 Thread GitBox


wgy8283335 commented on a change in pull request #10253:
URL: https://github.com/apache/shardingsphere/pull/10253#discussion_r629757456



##
File path: 
shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-oracle/src/main/antlr4/imports/oracle/DDLStatement.g4
##
@@ -1421,3 +1421,258 @@ leadCdbUriClause
 propertyClause
 : PROPERTY (SET | REMOVE) DEFAULT_CREDENTIAL EQ_ qualifiedCredentialName
 ;
+
+alterSystem
+: ALTER SYSTEM alterSystemOption
+;
+
+alterSystemOption
+: archiveLogClause
+| checkpointClause
+| checkDatafilesClause
+| distributedRecovClauses
+| flushClause
+| endSessionClauses
+| switchLogfileClause
+| suspendResumeClause
+| quiesceClauses
+| rollingMigrationClauses
+| rollingPatchClauses
+| alterSystemSecuriyClauses
+| affinityClauses
+| shutdownDispatcherClause
+| registerClause
+| setClause
+| resetClause
+| relocateClientClause
+| cancelSqlClause
+| flushPasswordfileMetadataCacheClause
+;
+
+archiveLogClause
+: ARCHIVE LOG instanceClause? (sequenceClause | changeClause | 
currentClause | groupClause | logfileClause | nextClause | allClause) 
toLocationClause?
+;
+
+checkpointClause
+: CHECKPOINT (GLOBAL | LOCAL)?
+;
+
+checkDatafilesClause
+: CHECK DATAFILES (GLOBAL | LOCAL)?
+;
+
+distributedRecovClauses
+: (ENABLE | DISABLE) DISTRIBUTED RECOVERY
+;
+
+flushClause
+: FLUSH flushClauseOption
+;
+
+endSessionClauses
+: (disconnectSessionClause | killSessionClause) (IMMEDIATE | NOREPLY)?
+;
+
+switchLogfileClause
+: SWITCH LOGFILE
+;
+
+suspendResumeClause
+: SUSPEND | RESUME
+;
+
+quiesceClauses
+: QUIESCE RESTRICTED | UNQUIESCE
+;
+
+rollingMigrationClauses
+: startRollingMigrationClause | stopRollingMigrationClause
+;
+
+rollingPatchClauses
+: startRollingPatchClause | stopRollingPatchClause
+;
+
+alterSystemSecuriyClauses
+: restrictedSessionClause | setEncryptionWalletOpenClause | 
setEncryptionWalletCloseClause | setEncryptionKeyClause
+;
+
+affinityClauses
+: enableAffinityClause | disableAffinityClause
+;
+
+shutdownDispatcherClause
+: SHUTDOWN IMMEDIATE? dispatcherName
+;
+
+registerClause
+: REGISTER
+;
+
+setClause
+: SET alterSystemSetClause+
+;
+
+resetClause
+: RESET alterSystemResetClause+
+;
+
+relocateClientClause
+: RELOCATE CLIENT clientId
+;
+
+cancelSqlClause
+: CANCEL SQL SQ_ sessionId serialNumber instanceId? sqlId? SQ_

Review comment:
   I think so.




-- 
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




[GitHub] [shardingsphere] wgy8283335 commented on a change in pull request #10253: Add SQL Definition for `ALTER SYSTEM` of Oracle Database

2021-05-10 Thread GitBox


wgy8283335 commented on a change in pull request #10253:
URL: https://github.com/apache/shardingsphere/pull/10253#discussion_r629757254



##
File path: 
shardingsphere-sql-parser/shardingsphere-sql-parser-dialect/shardingsphere-sql-parser-oracle/src/main/antlr4/imports/oracle/DDLStatement.g4
##
@@ -1421,3 +1421,258 @@ leadCdbUriClause
 propertyClause
 : PROPERTY (SET | REMOVE) DEFAULT_CREDENTIAL EQ_ qualifiedCredentialName
 ;
+
+alterSystem
+: ALTER SYSTEM alterSystemOption
+;
+
+alterSystemOption
+: archiveLogClause
+| checkpointClause
+| checkDatafilesClause
+| distributedRecovClauses
+| flushClause
+| endSessionClauses
+| switchLogfileClause
+| suspendResumeClause
+| quiesceClauses
+| rollingMigrationClauses
+| rollingPatchClauses
+| alterSystemSecuriyClauses
+| affinityClauses
+| shutdownDispatcherClause
+| registerClause
+| setClause
+| resetClause
+| relocateClientClause
+| cancelSqlClause
+| flushPasswordfileMetadataCacheClause
+;
+
+archiveLogClause
+: ARCHIVE LOG instanceClause? (sequenceClause | changeClause | 
currentClause | groupClause | logfileClause | nextClause | allClause) 
toLocationClause?
+;
+
+checkpointClause
+: CHECKPOINT (GLOBAL | LOCAL)?
+;
+
+checkDatafilesClause
+: CHECK DATAFILES (GLOBAL | LOCAL)?
+;
+
+distributedRecovClauses
+: (ENABLE | DISABLE) DISTRIBUTED RECOVERY
+;
+
+flushClause
+: FLUSH flushClauseOption
+;
+
+endSessionClauses
+: (disconnectSessionClause | killSessionClause) (IMMEDIATE | NOREPLY)?
+;
+
+switchLogfileClause
+: SWITCH LOGFILE
+;
+
+suspendResumeClause
+: SUSPEND | RESUME
+;
+
+quiesceClauses
+: QUIESCE RESTRICTED | UNQUIESCE
+;
+
+rollingMigrationClauses
+: startRollingMigrationClause | stopRollingMigrationClause
+;
+
+rollingPatchClauses
+: startRollingPatchClause | stopRollingPatchClause
+;
+
+alterSystemSecuriyClauses
+: restrictedSessionClause | setEncryptionWalletOpenClause | 
setEncryptionWalletCloseClause | setEncryptionKeyClause
+;
+
+affinityClauses
+: enableAffinityClause | disableAffinityClause
+;
+
+shutdownDispatcherClause
+: SHUTDOWN IMMEDIATE? dispatcherName
+;
+
+registerClause
+: REGISTER
+;
+
+setClause
+: SET alterSystemSetClause+
+;
+
+resetClause
+: RESET alterSystemResetClause+
+;
+
+relocateClientClause
+: RELOCATE CLIENT clientId
+;
+
+cancelSqlClause
+: CANCEL SQL SQ_ sessionId serialNumber instanceId? sqlId? SQ_
+;
+
+flushPasswordfileMetadataCacheClause
+: FLUSH PASSWORDFILE_METADATA_CACHE
+;
+
+instanceClause
+: INSTANCE instanceName
+;
+
+sequenceClause
+: SEQUENCE numberLiterals

Review comment:
   
![image](https://user-images.githubusercontent.com/22066046/117738618-0a3e6380-b22f-11eb-97bb-ddafa6206152.png)
   put INTEGER_ before NUMBER_
   
![image](https://user-images.githubusercontent.com/22066046/117738637-12969e80-b22f-11eb-9162-34e32f32c05a.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




[shardingsphere-benchmark] branch master updated: update benchmark result

2021-05-10 Thread zhangyonglun
This is an automated email from the ASF dual-hosted git repository.

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


The following commit(s) were added to refs/heads/master by this push:
 new 8f93a5b  update benchmark result
8f93a5b is described below

commit 8f93a5baa6b3b5327a74e6922f9834646c0f8850
Author: tuohai666 
AuthorDate: Tue May 11 08:00:23 2021 +0800

update benchmark result
---
 .../report/shardingsphere-benchmark-avg.xls| Bin 115200 -> 115200 bytes
 .../resources/report/shardingsphere-benchmark.xls  | Bin 88576 -> 89088 bytes
 2 files changed, 0 insertions(+), 0 deletions(-)

diff --git 
a/shardingsphere-benchmark/src/main/resources/report/shardingsphere-benchmark-avg.xls
 
b/shardingsphere-benchmark/src/main/resources/report/shardingsphere-benchmark-avg.xls
index a2f8b4a..067af79 100755
Binary files 
a/shardingsphere-benchmark/src/main/resources/report/shardingsphere-benchmark-avg.xls
 and 
b/shardingsphere-benchmark/src/main/resources/report/shardingsphere-benchmark-avg.xls
 differ
diff --git 
a/shardingsphere-benchmark/src/main/resources/report/shardingsphere-benchmark.xls
 
b/shardingsphere-benchmark/src/main/resources/report/shardingsphere-benchmark.xls
index b97bd8e..47b55c2 100755
Binary files 
a/shardingsphere-benchmark/src/main/resources/report/shardingsphere-benchmark.xls
 and 
b/shardingsphere-benchmark/src/main/resources/report/shardingsphere-benchmark.xls
 differ


[GitHub] [shardingsphere] wgy8283335 commented on pull request #10253: Add SQL Definition for `ALTER SYSTEM` of Oracle Database

2021-05-10 Thread GitBox


wgy8283335 commented on pull request #10253:
URL: https://github.com/apache/shardingsphere/pull/10253#issuecomment-837511286


   > Yes @tristaZero, I'll resolve that.
   
   It seems to have some conflicts.


-- 
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




[GitHub] [shardingsphere] TeslaCN opened a new pull request #10299: Correct PostgreSQL Proxy transaction state

2021-05-10 Thread GitBox


TeslaCN opened a new pull request #10299:
URL: https://github.com/apache/shardingsphere/pull/10299


   


-- 
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




[GitHub] [shardingsphere] TeslaCN opened a new issue #10298: Proxy PostgreSQL CLOSE_COMMAND not implemented

2021-05-10 Thread GitBox


TeslaCN opened a new issue #10298:
URL: https://github.com/apache/shardingsphere/issues/10298


   ## Feature Request
   
   ### Is your feature request related to a problem?
   
   
![image](https://user-images.githubusercontent.com/20503072/117676560-3bdc0e00-b1e0-11eb-8afd-722c06768957.png)
   
   ### Describe the feature you would like.
   Support CLOSE


-- 
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




[GitHub] [shardingsphere] codecov-commenter edited a comment on pull request #10295: Add GLOBAL_RULE_NODE in RegistryCenterNode,add global rule listener and event.

2021-05-10 Thread GitBox


codecov-commenter edited a comment on pull request #10295:
URL: https://github.com/apache/shardingsphere/pull/10295#issuecomment-836374777


   # 
[Codecov](https://codecov.io/gh/apache/shardingsphere/pull/10295?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#10295](https://codecov.io/gh/apache/shardingsphere/pull/10295?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (648a4c8) into 
[master](https://codecov.io/gh/apache/shardingsphere/commit/f08b09816901de8ea82f789b15328f5849db12c7?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (f08b098) will **increase** coverage by `0.36%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/shardingsphere/pull/10295/graphs/tree.svg?width=650=150=pr=ZvlXpWa7so_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/shardingsphere/pull/10295?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
   
   ```diff
   @@ Coverage Diff  @@
   ## master   #10295  +/-   ##
   
   + Coverage 67.94%   68.30%   +0.36% 
   - Complexity  683  684   +1 
   
 Files  1706 1707   +1 
 Lines 2929229194  -98 
 Branches   5270 5235  -35 
   
   + Hits  1990119940  +39 
   + Misses 7848 7711 -137 
 Partials   1543 1543  
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/shardingsphere/pull/10295?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...e/governance/core/registry/RegistryCenterNode.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtZ292ZXJuYW5jZS9zaGFyZGluZ3NwaGVyZS1nb3Zlcm5hbmNlLWNvcmUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL2dvdmVybmFuY2UvY29yZS9yZWdpc3RyeS9SZWdpc3RyeUNlbnRlck5vZGUuamF2YQ==)
 | `80.39% <100.00%> (+0.39%)` | `0.00 <0.00> (ø)` | |
   | 
[...istry/listener/impl/GlobalRuleChangedListener.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtZ292ZXJuYW5jZS9zaGFyZGluZ3NwaGVyZS1nb3Zlcm5hbmNlLWNvcmUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL2dvdmVybmFuY2UvY29yZS9yZWdpc3RyeS9saXN0ZW5lci9pbXBsL0dsb2JhbFJ1bGVDaGFuZ2VkTGlzdGVuZXIuamF2YQ==)
 | `100.00% <100.00%> (ø)` | `0.00 <0.00> (?)` | |
   | 
[...here/infra/binder/segment/table/TablesContext.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtaW5mcmEvc2hhcmRpbmdzcGhlcmUtaW5mcmEtYmluZGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGFyZGluZ3NwaGVyZS9pbmZyYS9iaW5kZXIvc2VnbWVudC90YWJsZS9UYWJsZXNDb250ZXh0LmphdmE=)
 | `63.33% <0.00%> (-6.94%)` | `0.00% <0.00%> (ø%)` | |
   | 
[...nfra/context/metadata/MetaDataContextsBuilder.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtaW5mcmEvc2hhcmRpbmdzcGhlcmUtaW5mcmEtY29udGV4dC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvaW5mcmEvY29udGV4dC9tZXRhZGF0YS9NZXRhRGF0YUNvbnRleHRzQnVpbGRlci5qYXZh)
 | `91.11% <0.00%> (-3.89%)` | `0.00% <0.00%> (ø%)` | |
   | 
[...ding/converter/ShardingRuleStatementConverter.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGFyZGluZ3NwaGVyZS9zaGFyZGluZy9jb252ZXJ0ZXIvU2hhcmRpbmdSdWxlU3RhdGVtZW50Q29udmVydGVyLmphdmE=)
 | `51.11% <0.00%> (-1.17%)` | `0.00% <0.00%> (ø%)` | |
   | 

[GitHub] [shardingsphere] fengying-wzm removed a comment on issue #10290: application run failed for 5.0.0-alpha

2021-05-10 Thread GitBox


fengying-wzm removed a comment on issue #10290:
URL: 
https://github.com/apache/shardingsphere/issues/10290#issuecomment-836642149


   我的放在Apollo,是properties格式,你转一下。
   
   ```properties
   
   # 是否启动读写分离
   spring.shardingsphere.enabled = true
   
   # 主库配置
   spring.shardingsphere.datasource.common.driver-class-name = 
com.mysql.cj.jdbc.Driver
   spring.shardingsphere.datasource.common.type = 
com.zaxxer.hikari.HikariDataSource
   
   spring.shardingsphere.datasource.names = master,slave0,slave1
   spring.shardingsphere.datasource.master.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.master.username = xxx
   spring.shardingsphere.datasource.master.password = xxx
   spring.shardingsphere.datasource.master.minimum-idle = 20
   spring.shardingsphere.datasource.master.maximum-pool-size = 50
   spring.shardingsphere.datasource.master.auto-commit = true
   spring.shardingsphere.datasource.master.idle-timeout = 3
   spring.shardingsphere.datasource.master.max-lifetime = 180
   spring.shardingsphere.datasource.master.connection-timeout = 3
   spring.shardingsphere.datasource.master.connection-test-query = SELECT 1
   
   # 从库slave0
   spring.shardingsphere.datasource.slave0.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.slave0.username = xxx
   spring.shardingsphere.datasource.slave0.password = xxx
   spring.shardingsphere.datasource.slave0.minimum-idle = 20
   spring.shardingsphere.datasource.slave0.maximum-pool-size = 50
   spring.shardingsphere.datasource.slave0.auto-commit = true
   spring.shardingsphere.datasource.slave0.idle-timeout = 3
   spring.shardingsphere.datasource.slave0.max-lifetime = 180
   spring.shardingsphere.datasource.slave0.connection-timeout = 3
   spring.shardingsphere.datasource.slave0.connection-test-query = SELECT 1
   
   # 从库slave1
   spring.shardingsphere.datasource.slave1.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.slave1.username = xxx
   spring.shardingsphere.datasource.slave1.password = xxx
   spring.shardingsphere.datasource.slave1.minimum-idle = 20
   spring.shardingsphere.datasource.slave1.maximum-pool-size = 50
   spring.shardingsphere.datasource.slave1.auto-commit = true
   spring.shardingsphere.datasource.slave1.idle-timeout = 3
   spring.shardingsphere.datasource.slave1.max-lifetime = 180
   spring.shardingsphere.datasource.slave1.connection-timeout = 3
   spring.shardingsphere.datasource.slave1.connection-test-query = SELECT 1
   
   # 负载均衡算法,默认轮询
   spring.shardingsphere.masterslave.load-balance-algorithm-type = round_robin
   spring.shardingsphere.masterslave.name = master-slave
   spring.shardingsphere.masterslave.master-data-source-name = master
   spring.shardingsphere.masterslave.slave-data-source-names = slave0,slave1
   
   # 打印mysql语句
   spring.shardingsphere.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




[GitHub] [shardingsphere] fengying-wzm commented on issue #10290: application run failed for 5.0.0-alpha

2021-05-10 Thread GitBox


fengying-wzm commented on issue #10290:
URL: 
https://github.com/apache/shardingsphere/issues/10290#issuecomment-836643930


   @zexho994 我的放在Apollo,是properties格式,你转一下。
   ```properties
   # 是否启动读写分离
   spring.shardingsphere.enabled = true
   
   # 主库配置
   spring.shardingsphere.datasource.common.driver-class-name = 
com.mysql.cj.jdbc.Driver
   spring.shardingsphere.datasource.common.type = 
com.zaxxer.hikari.HikariDataSource
   
   spring.shardingsphere.datasource.names = master,slave0,slave1
   spring.shardingsphere.datasource.master.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.master.username = xxx
   spring.shardingsphere.datasource.master.password = xxx
   spring.shardingsphere.datasource.master.minimum-idle = 20
   spring.shardingsphere.datasource.master.maximum-pool-size = 50
   spring.shardingsphere.datasource.master.auto-commit = true
   spring.shardingsphere.datasource.master.idle-timeout = 3
   spring.shardingsphere.datasource.master.max-lifetime = 180
   spring.shardingsphere.datasource.master.connection-timeout = 3
   spring.shardingsphere.datasource.master.connection-test-query = SELECT 1
   
   # 从库slave0
   spring.shardingsphere.datasource.slave0.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.slave0.username = xxx
   spring.shardingsphere.datasource.slave0.password = xxx
   spring.shardingsphere.datasource.slave0.minimum-idle = 20
   spring.shardingsphere.datasource.slave0.maximum-pool-size = 50
   spring.shardingsphere.datasource.slave0.auto-commit = true
   spring.shardingsphere.datasource.slave0.idle-timeout = 3
   spring.shardingsphere.datasource.slave0.max-lifetime = 180
   spring.shardingsphere.datasource.slave0.connection-timeout = 3
   spring.shardingsphere.datasource.slave0.connection-test-query = SELECT 1
   
   # 从库slave1
   spring.shardingsphere.datasource.slave1.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.slave1.username = xxx
   spring.shardingsphere.datasource.slave1.password = xxx
   spring.shardingsphere.datasource.slave1.minimum-idle = 20
   spring.shardingsphere.datasource.slave1.maximum-pool-size = 50
   spring.shardingsphere.datasource.slave1.auto-commit = true
   spring.shardingsphere.datasource.slave1.idle-timeout = 3
   spring.shardingsphere.datasource.slave1.max-lifetime = 180
   spring.shardingsphere.datasource.slave1.connection-timeout = 3
   spring.shardingsphere.datasource.slave1.connection-test-query = SELECT 1
   
   # 负载均衡算法,默认轮询
   spring.shardingsphere.masterslave.load-balance-algorithm-type = round_robin
   spring.shardingsphere.masterslave.name = master-slave
   spring.shardingsphere.masterslave.master-data-source-name = master
   spring.shardingsphere.masterslave.slave-data-source-names = slave0,slave1
   
   # 打印mysql语句
   spring.shardingsphere.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




[GitHub] [shardingsphere] fengying-wzm commented on issue #10290: application run failed for 5.0.0-alpha

2021-05-10 Thread GitBox


fengying-wzm commented on issue #10290:
URL: 
https://github.com/apache/shardingsphere/issues/10290#issuecomment-836642149


   我的放在Apollo,是properties格式,你转一下。
   
   ```properties
   
   # 是否启动读写分离
   spring.shardingsphere.enabled = true
   
   # 主库配置
   spring.shardingsphere.datasource.common.driver-class-name = 
com.mysql.cj.jdbc.Driver
   spring.shardingsphere.datasource.common.type = 
com.zaxxer.hikari.HikariDataSource
   
   spring.shardingsphere.datasource.names = master,slave0,slave1
   spring.shardingsphere.datasource.master.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.master.username = xxx
   spring.shardingsphere.datasource.master.password = xxx
   spring.shardingsphere.datasource.master.minimum-idle = 20
   spring.shardingsphere.datasource.master.maximum-pool-size = 50
   spring.shardingsphere.datasource.master.auto-commit = true
   spring.shardingsphere.datasource.master.idle-timeout = 3
   spring.shardingsphere.datasource.master.max-lifetime = 180
   spring.shardingsphere.datasource.master.connection-timeout = 3
   spring.shardingsphere.datasource.master.connection-test-query = SELECT 1
   
   # 从库slave0
   spring.shardingsphere.datasource.slave0.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.slave0.username = xxx
   spring.shardingsphere.datasource.slave0.password = xxx
   spring.shardingsphere.datasource.slave0.minimum-idle = 20
   spring.shardingsphere.datasource.slave0.maximum-pool-size = 50
   spring.shardingsphere.datasource.slave0.auto-commit = true
   spring.shardingsphere.datasource.slave0.idle-timeout = 3
   spring.shardingsphere.datasource.slave0.max-lifetime = 180
   spring.shardingsphere.datasource.slave0.connection-timeout = 3
   spring.shardingsphere.datasource.slave0.connection-test-query = SELECT 1
   
   # 从库slave1
   spring.shardingsphere.datasource.slave1.jdbcUrl = 
jdbc:mysql://xxx:3306/xxx?useUnicode=true=utf-8=false=GMT%2B8
   spring.shardingsphere.datasource.slave1.username = xxx
   spring.shardingsphere.datasource.slave1.password = xxx
   spring.shardingsphere.datasource.slave1.minimum-idle = 20
   spring.shardingsphere.datasource.slave1.maximum-pool-size = 50
   spring.shardingsphere.datasource.slave1.auto-commit = true
   spring.shardingsphere.datasource.slave1.idle-timeout = 3
   spring.shardingsphere.datasource.slave1.max-lifetime = 180
   spring.shardingsphere.datasource.slave1.connection-timeout = 3
   spring.shardingsphere.datasource.slave1.connection-test-query = SELECT 1
   
   # 负载均衡算法,默认轮询
   spring.shardingsphere.masterslave.load-balance-algorithm-type = round_robin
   spring.shardingsphere.masterslave.name = master-slave
   spring.shardingsphere.masterslave.master-data-source-name = master
   spring.shardingsphere.masterslave.slave-data-source-names = slave0,slave1
   
   # 打印mysql语句
   spring.shardingsphere.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




[GitHub] [shardingsphere] zexho994 commented on issue #10290: application run failed for 5.0.0-alpha

2021-05-10 Thread GitBox


zexho994 commented on issue #10290:
URL: 
https://github.com/apache/shardingsphere/issues/10290#issuecomment-836623209


   @fengying-wzm  请问下datasource你是怎么配置的?


-- 
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




[GitHub] [shardingsphere] MingxingLAI commented on issue #8982: show tables like xxx% can not return correct result

2021-05-10 Thread GitBox


MingxingLAI commented on issue #8982:
URL: https://github.com/apache/shardingsphere/issues/8982#issuecomment-836611701


   I tested the code and it seems to be able to return the correct result. Can 
you give a test case?  @tristaZero @strongduanmu 


-- 
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




[GitHub] [shardingsphere] fengying-wzm commented on issue #10290: application run failed for 5.0.0-alpha

2021-05-10 Thread GitBox


fengying-wzm commented on issue #10290:
URL: 
https://github.com/apache/shardingsphere/issues/10290#issuecomment-836610718


   5.0.0官网给的配置有点坑爹,我的如下通过:
   
   ```yaml
   spring:
 shardingsphere:
   rules:
 encrypt:
   encryptors:
 aes-encryptor:
   props:
 aes-key-value: 123456abc
   type: aes
   tables:
 user_info:
   columns:
 name:
   cipherColumn: name_cipher
   encryptorName: aes-encryptor
   plainColumn: name
   
   ```
   


-- 
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




[GitHub] [shardingsphere] taojintianxia closed issue #10296: got an error `Your password does not satisfy the current policy requirements` on proxy

2021-05-10 Thread GitBox


taojintianxia closed issue #10296:
URL: https://github.com/apache/shardingsphere/issues/10296


   


-- 
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




[shardingsphere] branch master updated: Change authority configuration (#10297)

2021-05-10 Thread duanzhengqiang
This is an automated email from the ASF dual-hosted git repository.

duanzhengqiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
 new 064a8cb  Change authority configuration (#10297)
064a8cb is described below

commit 064a8cb15a2204641210213ded80bdc0af268382
Author: Juan Pan(Trista) 
AuthorDate: Mon May 10 19:09:20 2021 +0800

Change authority configuration (#10297)
---
 .../src/main/resources/conf/server.yaml| 14 -
 .../src/main/resources/conf/server.yaml| 23 +++---
 2 files changed, 20 insertions(+), 17 deletions(-)

diff --git 
a/shardingsphere-distribution/shardingsphere-proxy-distribution/src/main/resources/conf/server.yaml
 
b/shardingsphere-distribution/shardingsphere-proxy-distribution/src/main/resources/conf/server.yaml
index 9ddd6b6..cda98cd 100644
--- 
a/shardingsphere-distribution/shardingsphere-proxy-distribution/src/main/resources/conf/server.yaml
+++ 
b/shardingsphere-distribution/shardingsphere-proxy-distribution/src/main/resources/conf/server.yaml
@@ -20,7 +20,7 @@
 # If you want to configure governance, authorization and proxy properties, 
please refer to this file.
 # 
 
##
-#
+
 #governance:
 #  name: governance_ds
 #  registryCenter:
@@ -33,14 +33,18 @@
 #  operationTimeoutMilliseconds: 500
 #  overwrite: false
 
-#users:
-#  - root@%:root
-#  - sharding@:sharding
-
 #scaling:
 #  blockQueueSize: 1
 #  workerThread: 40
 
+#rules:
+#  - !AUTHORITY
+#users:
+#  - root@%:root
+#  - sharding@:sharding
+#provider:
+#  type: NATIVE
+
 #props:
 #  max-connections-size-per-query: 1
 #  executor-size: 16  # Infinite by default.
diff --git 
a/shardingsphere-proxy/shardingsphere-proxy-bootstrap/src/main/resources/conf/server.yaml
 
b/shardingsphere-proxy/shardingsphere-proxy-bootstrap/src/main/resources/conf/server.yaml
index af6ebc5..cda98cd 100644
--- 
a/shardingsphere-proxy/shardingsphere-proxy-bootstrap/src/main/resources/conf/server.yaml
+++ 
b/shardingsphere-proxy/shardingsphere-proxy-bootstrap/src/main/resources/conf/server.yaml
@@ -20,7 +20,7 @@
 # If you want to configure governance, authorization and proxy properties, 
please refer to this file.
 # 
 
##
-#
+
 #governance:
 #  name: governance_ds
 #  registryCenter:
@@ -32,15 +32,19 @@
 #  maxRetries: 3
 #  operationTimeoutMilliseconds: 500
 #  overwrite: false
-#
-#users:
-#  - root@%:root
-#  - sharding@:sharding
-#
+
 #scaling:
 #  blockQueueSize: 1
 #  workerThread: 40
-#
+
+#rules:
+#  - !AUTHORITY
+#users:
+#  - root@%:root
+#  - sharding@:sharding
+#provider:
+#  type: NATIVE
+
 #props:
 #  max-connections-size-per-query: 1
 #  executor-size: 16  # Infinite by default.
@@ -56,8 +60,3 @@
 #  sql-show: false
 #  check-table-metadata-enabled: false
 #  lock-wait-timeout-milliseconds: 5 # The maximum time to wait for a lock
-#
-#rules:
-#  - !AUTHORITY
-#provider:
-#  type: NATIVE


[GitHub] [shardingsphere] strongduanmu merged pull request #10297: Change authority configuration

2021-05-10 Thread GitBox


strongduanmu merged pull request #10297:
URL: https://github.com/apache/shardingsphere/pull/10297


   


-- 
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




[GitHub] [shardingsphere] tristaZero opened a new pull request #10297: Change authority configuration

2021-05-10 Thread GitBox


tristaZero opened a new pull request #10297:
URL: https://github.com/apache/shardingsphere/pull/10297


   #10251 


-- 
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




[GitHub] [shardingsphere-elasticjob] skaic opened a new issue #1891: add Tracing Api

2021-05-10 Thread GitBox


skaic opened a new issue #1891:
URL: https://github.com/apache/shardingsphere-elasticjob/issues/1891


   ## Question
   
   I want have a Tracing Api , and I can use it to show the Tracing log in my 
admin page.
   
   


-- 
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




[GitHub] [shardingsphere] taojintianxia edited a comment on issue #10296: got an error `Your password does not satisfy the current policy requirements` on proxy

2021-05-10 Thread GitBox


taojintianxia edited a comment on issue #10296:
URL: 
https://github.com/apache/shardingsphere/issues/10296#issuecomment-836493961






-- 
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




[GitHub] [shardingsphere] taojintianxia commented on issue #10296: got an error `Your password does not satisfy the current policy requirements` on proxy

2021-05-10 Thread GitBox


taojintianxia commented on issue #10296:
URL: 
https://github.com/apache/shardingsphere/issues/10296#issuecomment-836493961


   but when connect to proxy from another machine , execute any sql will get 
follwoing error : 
   
   ```log
   Unknown exception: [SQL checking failed. Error message: .]
   ``
   
   
![image](https://user-images.githubusercontent.com/4112856/117641771-230c3200-b1b9-11eb-9c6e-12d54f2000c6.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




[GitHub] [shardingsphere] taojintianxia edited a comment on issue #10296: got an error `Your password does not satisfy the current policy requirements` on proxy

2021-05-10 Thread GitBox


taojintianxia edited a comment on issue #10296:
URL: 
https://github.com/apache/shardingsphere/issues/10296#issuecomment-836469810


   on mysql machine, launch mysql and execute the command :
   ```mysql
   >mysql uninstall plugin validate_password;
   ```
   
   after that clause , proxy could start up.


-- 
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




[GitHub] [shardingsphere-elasticjob] skaic commented on issue #1865: springboot2.x+ elasticjo3.0.0-RC1 +mybatis-dynamic3.3.0, can not work well with elasticjob.tracing.type=RDB

2021-05-10 Thread GitBox


skaic commented on issue #1865:
URL: 
https://github.com/apache/shardingsphere-elasticjob/issues/1865#issuecomment-836478312


   you can register a bean of TracingConfiguration\ like this:
   
   ```
   @Setter
   @Configuration
   @ConfigurationProperties(prefix = "elasticjob.tracing.datasource")
   public class ElasticJobTracingConfiguration {
   
   private String driverClassName;
   
   private String url;
   
   private String username;
   
   private String password;
   
   @Bean
   public TracingConfiguration myTracingConfiguration() {
   BasicDataSource dataSource = new BasicDataSource();
   dataSource.setDriverClassName(driverClassName);
   dataSource.setUrl(url);
   dataSource.setUsername(username);
   dataSource.setPassword(password);
   return new TracingConfiguration<>("RDB", dataSource);
   }
   
   }
   ```
   
   and 
   
   ```
   elasticjob:
 tracing:
   datasource:
 driverClassName: com.mysql.jdbc.Driver
 url: 
jdbc:mysql:///databases?useUnicode=true=UTF-8=false
 username: root
 password: 
   ```


-- 
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




[GitHub] [shardingsphere] taojintianxia commented on issue #10296: got an error `Your password does not satisfy the current policy requirements` on proxy

2021-05-10 Thread GitBox


taojintianxia commented on issue #10296:
URL: 
https://github.com/apache/shardingsphere/issues/10296#issuecomment-836469810


   on mysql machine, launch mysql
   ```mysql
   >mysql uninstall plugin validate_password;
   ```
   
   after that clause , the problem solved


-- 
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




[GitHub] [shardingsphere] taojintianxia opened a new issue #10296: got an error `Your password does not satisfy the current policy requirements` on proxy

2021-05-10 Thread GitBox


taojintianxia opened a new issue #10296:
URL: https://github.com/apache/shardingsphere/issues/10296


   let's suppose there are 3 centos 7 to reproduce the issue
   
   proxy : code from github @ 2021-05-10 : 17:26
   mysql : 5.7.34 MySQL Community Server
   
   |  machine-A   | machine-B  | machine-C  | 
   |    |   |   |
   | proxy  | mysql-A | mysql-B |
   
   I could access MySQL-A and MySQL-B seperately from proxy by mysql client.
   
   but when I try to start up the proxy , I got following error : 
   
   ```log
   Exception in thread "main" java.lang.IllegalStateException: Error while 
loading privilege with java.util.concurrent.FutureTask@f73dcd6
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.lambda$save$3(StoragePrivilegeBuilder.java:111)
at java.lang.Iterable.forEach(Iterable.java:75)
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.save(StoragePrivilegeBuilder.java:107)
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.buildPrivilegesInStorage(StoragePrivilegeBuilder.java:94)
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.lambda$buildPrivilegesInStorage$1(StoragePrivilegeBuilder.java:84)
at 
java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
at 
java.util.LinkedList$LLSpliterator.forEachRemaining(LinkedList.java:1235)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:482)
at 
java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:472)
at 
java.util.stream.ForEachOps$ForEachOp.evaluateSequential(ForEachOps.java:150)
at 
java.util.stream.ForEachOps$ForEachOp$OfRef.evaluateSequential(ForEachOps.java:173)
at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
at 
java.util.stream.ReferencePipeline.forEach(ReferencePipeline.java:485)
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.buildPrivilegesInStorage(StoragePrivilegeBuilder.java:84)
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.build(StoragePrivilegeBuilder.java:71)
at 
org.apache.shardingsphere.authority.provider.natived.NativeAuthorityProviderAlgorithm.init(NativeAuthorityProviderAlgorithm.java:42)
at 
org.apache.shardingsphere.authority.rule.AuthorityRule.(AuthorityRule.java:48)
at 
org.apache.shardingsphere.authority.rule.builder.AuthorityRuleBuilder.build(AuthorityRuleBuilder.java:36)
at 
org.apache.shardingsphere.authority.rule.builder.AuthorityRuleBuilder.build(AuthorityRuleBuilder.java:32)
at 
org.apache.shardingsphere.infra.rule.builder.ShardingSphereRulesBuilder.buildGlobalRules(ShardingSphereRulesBuilder.java:100)
at 
org.apache.shardingsphere.infra.context.metadata.MetaDataContextsBuilder.buildGlobalSchemaMetaData(MetaDataContextsBuilder.java:109)
at 
org.apache.shardingsphere.infra.context.metadata.MetaDataContextsBuilder.build(MetaDataContextsBuilder.java:96)
at 
org.apache.shardingsphere.proxy.initializer.impl.AbstractBootstrapInitializer.createMetaDataContexts(AbstractBootstrapInitializer.java:84)
at 
org.apache.shardingsphere.proxy.initializer.impl.AbstractBootstrapInitializer.init(AbstractBootstrapInitializer.java:67)
at org.apache.shardingsphere.proxy.Bootstrap.main(Bootstrap.java:48)
   Caused by: java.util.concurrent.ExecutionException: 
org.apache.shardingsphere.infra.exception.ShardingSphereException: 
java.sql.SQLException: Your password does not satisfy the current policy 
requirements
at java.util.concurrent.FutureTask.report(FutureTask.java:122)
at java.util.concurrent.FutureTask.get(FutureTask.java:206)
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.lambda$save$3(StoragePrivilegeBuilder.java:109)
... 24 more
   Caused by: 
org.apache.shardingsphere.infra.exception.ShardingSphereException: 
java.sql.SQLException: Your password does not satisfy the current policy 
requirements
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.save(StoragePrivilegeBuilder.java:125)
at 
org.apache.shardingsphere.authority.provider.natived.builder.StoragePrivilegeBuilder.lambda$save$2(StoragePrivilegeBuilder.java:105)
at 
java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:511)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at 
java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
at 
java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
at java.lang.Thread.run(Thread.java:748)
   Caused by: java.sql.SQLException: Your password does not satisfy the current 
policy 

[shardingsphere] branch master updated: Rename name config item form yaml for Readwrite-splitting (#10294)

2021-05-10 Thread panjuan
This is an automated email from the ASF dual-hosted git repository.

panjuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
 new 3ce918e  Rename name config item form yaml for Readwrite-splitting 
(#10294)
3ce918e is described below

commit 3ce918ed0b91242e707dfd2bf74269b1755f76fd
Author: Liang Zhang 
AuthorDate: Mon May 10 16:52:40 2021 +0800

Rename name config item form yaml for Readwrite-splitting (#10294)
---
 .../content/features/test-engine/performance-test-sysbench.cn.md | 6 ++
 .../content/features/test-engine/performance-test-sysbench.en.md | 6 ++
 .../document/content/features/test-engine/performance-test.cn.md | 3 +--
 .../document/content/features/test-engine/performance-test.en.md | 9 -
 .../user-manual/shardingsphere-jdbc/configuration/yaml/mix.cn.md | 5 ++---
 .../user-manual/shardingsphere-jdbc/configuration/yaml/mix.en.md | 5 ++---
 .../resources/META-INF/zookeeper/local/readwrite-splitting.yaml  | 1 -
 .../src/main/resources/META-INF/hint-write-only.yaml | 1 -
 .../resources/META-INF/readwrite-splitting-shadow-databases.yaml | 1 -
 .../src/main/resources/META-INF/readwrite-splitting.yaml | 1 -
 .../src/main/resources/conf/config-readwrite-splitting.yaml  | 7 +++
 .../src/main/resources/conf/config-write-only.yaml   | 1 -
 .../src/main/resources/conf/config-readwrite-splitting.yaml  | 2 --
 .../rule/YamlReadwriteSplittingDataSourceRuleConfiguration.java  | 2 --
 .../converter/AlterReadwriteSplittingRuleStatementConverter.java | 1 -
 .../CreateReadwriteSplittingRuleStatementConverter.java  | 1 -
 ...teSplittingRuleAlgorithmProviderConfigurationYamlSwapper.java | 1 -
 .../swapper/ReadwriteSplittingRuleConfigurationYamlSwapper.java  | 1 -
 ...littingRuleAlgorithmProviderConfigurationYamlSwapperTest.java | 1 -
 .../ReadwriteSplittingRuleConfigurationYamlSwapperTest.java  | 3 ---
 .../src/test/resources/yaml/readwrite-splitting-rule.yaml| 1 -
 .../yaml/registryCenter/data-readwrite-splitting-rule.yaml   | 1 -
 .../src/test/resources/config/config-calcite.yaml| 1 -
 .../src/test/resources/config/config-readwrite-splitting.yaml| 1 -
 .../configWithReadwriteSplittingDataSourceWithProps.yaml | 1 -
 .../configWithReadwriteSplittingDataSourceWithoutProps.yaml  | 1 -
 .../src/test/resources/yaml/readwrite-splitting-rule.yaml| 1 -
 .../rdl/impl/AlterReadwriteSplittingRuleBackendHandler.java  | 1 -
 .../src/main/resources/conf/config-readwrite-splitting.yaml  | 3 ---
 .../proxy/config/ProxyConfigurationLoaderTest.java   | 1 -
 .../src/test/resources/conf/config-readwrite-splitting.yaml  | 1 -
 .../proxy/conf/config-readwrite-splitting.yaml   | 1 -
 .../src/test/resources/env/readwrite_splitting/rules.yaml| 1 -
 33 files changed, 16 insertions(+), 57 deletions(-)

diff --git 
a/docs/document/content/features/test-engine/performance-test-sysbench.cn.md 
b/docs/document/content/features/test-engine/performance-test-sysbench.cn.md
index c3e6b63..684d349 100644
--- a/docs/document/content/features/test-engine/performance-test-sysbench.cn.md
+++ b/docs/document/content/features/test-engine/performance-test-sysbench.cn.md
@@ -520,10 +520,9 @@ dataSources:
 maintenanceIntervalMilliseconds: 3
 
 rules:
-- !REPLICA_QUERY
+- !READWRITE_SPLITTING
   dataSources:
 pr_ds:
-  name: pr_ds
   primaryDataSourceName: ds_0
   replicaDataSourceNames:
 - ds_0
@@ -707,10 +706,9 @@ rules:
   props:
 worker-id: 123
 
-- !REPLICA_QUERY
+- !READWRITE_SPLITTING
   dataSources:
 ds_0:
-  name: ds_0
   primaryDataSourceName: primary_ds_0
   replicaDataSourceNames:
 - primary_ds_0
diff --git 
a/docs/document/content/features/test-engine/performance-test-sysbench.en.md 
b/docs/document/content/features/test-engine/performance-test-sysbench.en.md
index 0dcb8a3..5c4400b 100644
--- a/docs/document/content/features/test-engine/performance-test-sysbench.en.md
+++ b/docs/document/content/features/test-engine/performance-test-sysbench.en.md
@@ -520,10 +520,9 @@ dataSources:
 maintenanceIntervalMilliseconds: 3
 
 rules:
-- !REPLICA_QUERY
+- !READWRITE_SPLITTING
   dataSources:
 pr_ds:
-  name: pr_ds
   primaryDataSourceName: ds_0
   replicaDataSourceNames:
 - ds_0
@@ -707,10 +706,9 @@ rules:
   props:
 worker-id: 123
 
-- !REPLICA_QUERY
+- !READWRITE_SPLITTING
   dataSources:
 ds_0:
-  name: ds_0
   primaryDataSourceName: primary_ds_0
   replicaDataSourceNames:
 - primary_ds_0
diff --git a/docs/document/content/features/test-engine/performance-test.cn.md 
b/docs/document/content/features/test-engine/performance-test.cn.md
index 951c2a6..ddc9cb4 100644
--- 

[GitHub] [shardingsphere] tristaZero merged pull request #10294: Rename name config item form yaml for Readwrite-splitting

2021-05-10 Thread GitBox


tristaZero merged pull request #10294:
URL: https://github.com/apache/shardingsphere/pull/10294


   


-- 
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




[GitHub] [shardingsphere] codecov-commenter commented on pull request #10295: Add GLOBAL_RULE_NODE in RegistryCenterNode.

2021-05-10 Thread GitBox


codecov-commenter commented on pull request #10295:
URL: https://github.com/apache/shardingsphere/pull/10295#issuecomment-836374777


   # 
[Codecov](https://codecov.io/gh/apache/shardingsphere/pull/10295?src=pr=h1_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 Report
   > Merging 
[#10295](https://codecov.io/gh/apache/shardingsphere/pull/10295?src=pr=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (562e7e6) into 
[master](https://codecov.io/gh/apache/shardingsphere/commit/f08b09816901de8ea82f789b15328f5849db12c7?el=desc_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 (f08b098) will **increase** coverage by `0.35%`.
   > The diff coverage is `100.00%`.
   
   [![Impacted file tree 
graph](https://codecov.io/gh/apache/shardingsphere/pull/10295/graphs/tree.svg?width=650=150=pr=ZvlXpWa7so_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)](https://codecov.io/gh/apache/shardingsphere/pull/10295?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
   
   ```diff
   @@ Coverage Diff  @@
   ## master   #10295  +/-   ##
   
   + Coverage 67.94%   68.29%   +0.35% 
   - Complexity  683  684   +1 
   
 Files  1706 1706  
 Lines 2929229196  -96 
 Branches   5270 5235  -35 
   
   + Hits  1990119939  +38 
   + Misses 7848 7714 -134 
 Partials   1543 1543  
   ```
   
   
   | [Impacted 
Files](https://codecov.io/gh/apache/shardingsphere/pull/10295?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation)
 | Coverage Δ | Complexity Δ | |
   |---|---|---|---|
   | 
[...e/governance/core/registry/RegistryCenterNode.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtZ292ZXJuYW5jZS9zaGFyZGluZ3NwaGVyZS1nb3Zlcm5hbmNlLWNvcmUvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3NoYXJkaW5nc3BoZXJlL2dvdmVybmFuY2UvY29yZS9yZWdpc3RyeS9SZWdpc3RyeUNlbnRlck5vZGUuamF2YQ==)
 | `80.39% <100.00%> (+0.39%)` | `0.00 <0.00> (ø)` | |
   | 
[...here/infra/binder/segment/table/TablesContext.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtaW5mcmEvc2hhcmRpbmdzcGhlcmUtaW5mcmEtYmluZGVyL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGFyZGluZ3NwaGVyZS9pbmZyYS9iaW5kZXIvc2VnbWVudC90YWJsZS9UYWJsZXNDb250ZXh0LmphdmE=)
 | `63.33% <0.00%> (-6.94%)` | `0.00% <0.00%> (ø%)` | |
   | 
[...nfra/context/metadata/MetaDataContextsBuilder.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtaW5mcmEvc2hhcmRpbmdzcGhlcmUtaW5mcmEtY29udGV4dC9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hhcmRpbmdzcGhlcmUvaW5mcmEvY29udGV4dC9tZXRhZGF0YS9NZXRhRGF0YUNvbnRleHRzQnVpbGRlci5qYXZh)
 | `91.11% <0.00%> (-3.89%)` | `0.00% <0.00%> (ø%)` | |
   | 
[...ding/converter/ShardingRuleStatementConverter.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtZmVhdHVyZXMvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmcvc2hhcmRpbmdzcGhlcmUtc2hhcmRpbmctY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGFyZGluZ3NwaGVyZS9zaGFyZGluZy9jb252ZXJ0ZXIvU2hhcmRpbmdSdWxlU3RhdGVtZW50Q29udmVydGVyLmphdmE=)
 | `51.11% <0.00%> (-1.17%)` | `0.00% <0.00%> (ø%)` | |
   | 
[...ig/yaml/swapper/YamlProxyConfigurationSwapper.java](https://codecov.io/gh/apache/shardingsphere/pull/10295/diff?src=pr=tree_medium=referral_source=github_content=comment_campaign=pr+comments_term=The+Apache+Software+Foundation#diff-c2hhcmRpbmdzcGhlcmUtcHJveHkvc2hhcmRpbmdzcGhlcmUtcHJveHktY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGFyZGluZ3NwaGVyZS9wcm94eS9jb25maWcveWFtbC9zd2FwcGVyL1lhbWxQcm94eUNvbmZpZ3VyYXRpb25Td2FwcGVyLmphdmE=)
 | `91.66% <0.00%> (-0.65%)` | `1.00% <0.00%> (ø%)` | |
   | 

[GitHub] [shardingsphere] zhujunxxxxx commented on a change in pull request #10287: Move the user configuration to authority rule

2021-05-10 Thread GitBox


zhujunx commented on a change in pull request #10287:
URL: https://github.com/apache/shardingsphere/pull/10287#discussion_r629153249



##
File path: 
shardingsphere-infra/shardingsphere-infra-context/src/main/java/org/apache/shardingsphere/infra/context/metadata/MetaDataContextsBuilder.java
##
@@ -94,7 +93,7 @@ public StandardMetaDataContexts build() throws SQLException {
 for (String each : schemaRuleConfigs.keySet()) {
 mataDataMap.put(each, buildMetaData(each));
 }
-return new StandardMetaDataContexts(mataDataMap, 
buildGlobalSchemaMetaData(mataDataMap), executorEngine, new 
ShardingSphereUsers(users), props);
+return new StandardMetaDataContexts(mataDataMap, 
buildGlobalSchemaMetaData(mataDataMap), executorEngine, new 
ShardingSphereUsers(getUsersFromAuthorityRule()), props);

Review comment:
   @tristaZero 
   > ShardingSphereUsers(users) is expected to be removed from 
StandardMetaDataContexts
   it's need remove? Ok, next PR I will remove it.
   




-- 
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




[GitHub] [shardingsphere] zexho994 commented on issue #10290: application run failed for spring-boot-starter 5.0

2021-05-10 Thread GitBox


zexho994 commented on issue #10290:
URL: 
https://github.com/apache/shardingsphere/issues/10290#issuecomment-836358628


   ## update
   spring boot version : 2.1.7.RELEASE
   ss version : 5.0.0-alpha
   
   updated dependency 
   ```
   
   org.apache.shardingsphere
   shardingsphere-jdbc-core-spring-boot-starter
   5.0.0-alpha
   
   ```
   
   updated application.yml
   ```
   spring:
 shardingsphere:
   rules:
 encrypt:
   tables:
 source_files:
   columns:
 file_name:
   cipher-column: file_name
   encryptor-name: aes-encryptor
   encryptors:
 aes-encryptor:
   type: AES
   props:
 aes-key-value: 123456df
   ```
   
   but...  now there is a new error
   ```
   [2021-05-10 
16:18:57.468][WARN][main][org.springframework.context.support.AbstractApplicationContext:557][]:
 Exception encountered during context initialization - cancelling refresh 
attempt: org.springframework.beans.factory.BeanCreationException: Error 
creating bean with name 
'org.apache.shardingsphere.spring.boot.SpringBootConfiguration': Initialization 
of bean failed; nested exception is java.lang.reflect.InvocationTargetException
   [2021-05-10 
16:18:57.537][ERROR][main][org.springframework.boot.SpringApplication:822][]: 
Application run failed
   org.springframework.beans.factory.BeanCreationException: Error creating bean 
with name 'org.apache.shardingsphere.spring.boot.SpringBootConfiguration': 
Initialization of bean failed; nested exception is 
java.lang.reflect.InvocationTargetException
at 
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:601)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:515)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractBeanFactory.lambda$doGetBean$0(AbstractBeanFactory.java:320)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:222)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:318)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:199)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.ConstructorResolver.instantiateUsingFactoryMethod(ConstructorResolver.java:392)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.instantiateUsingFactoryMethod(AbstractAutowireCapableBeanFactory.java:1321)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBeanInstance(AbstractAutowireCapableBeanFactory.java:1160)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.doCreateBean(AbstractAutowireCapableBeanFactory.java:555)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractAutowireCapableBeanFactory.createBean(AbstractAutowireCapableBeanFactory.java:515)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractBeanFactory.lambda$doGetBean$0(AbstractBeanFactory.java:320)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.DefaultSingletonBeanRegistry.getSingleton(DefaultSingletonBeanRegistry.java:222)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractBeanFactory.doGetBean(AbstractBeanFactory.java:318)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.beans.factory.support.AbstractBeanFactory.getBean(AbstractBeanFactory.java:204)
 ~[spring-beans-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.context.support.PostProcessorRegistrationDelegate.registerBeanPostProcessors(PostProcessorRegistrationDelegate.java:228)
 ~[spring-context-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.context.support.AbstractApplicationContext.registerBeanPostProcessors(AbstractApplicationContext.java:721)
 ~[spring-context-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 
org.springframework.context.support.AbstractApplicationContext.refresh(AbstractApplicationContext.java:534)
 ~[spring-context-5.1.9.RELEASE.jar:5.1.9.RELEASE]
at 

[GitHub] [shardingsphere] huanghao495430759 opened a new pull request #10295: Add GLOBAL_RULE_NODE in RegistryCenterNode.

2021-05-10 Thread GitBox


huanghao495430759 opened a new pull request #10295:
URL: https://github.com/apache/shardingsphere/pull/10295


   Fixes #10252.
   - Add GLOBAL_RULE_NODE in RegistryCenterNode.
   


-- 
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




[shardingsphere] branch master updated: Move the user configuration to authority rule (#10287)

2021-05-10 Thread panjuan
This is an automated email from the ASF dual-hosted git repository.

panjuan pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/shardingsphere.git


The following commit(s) were added to refs/heads/master by this push:
 new 55b219a  Move the user configuration to authority rule (#10287)
55b219a is described below

commit 55b219a9a2d337be454f71f45951b6a87974c727
Author: Zhu jun 
AuthorDate: Mon May 10 15:08:46 2021 +0800

Move the user configuration to authority rule (#10287)

* move the user configuration to authority rule

* adapter config

* adapter config

* fix test

* fix test

* fix yaml
---
 .../sharding-proxy/sharding/conf/server.yaml   |  8 +--
 .../src/main/resources/conf/server.yaml| 10 ++---
 .../src/main/resources/conf/server.yaml|  8 +--
 .../metadata/GovernanceMetaDataContexts.java   |  6 ++---
 .../governance/core/facade/GovernanceFacade.java   |  6 ++---
 .../governance/core/registry/RegistryCenter.java   | 10 +
 .../core/facade/GovernanceFacadeTest.java  |  6 ++---
 .../core/registry/RegistryCenterTest.java  |  3 +--
 .../api/config/AuthorityRuleConfiguration.java |  7 +-
 .../rule/builder/AuthorityRuleBuilder.java |  6 ++---
 .../DefaultAuthorityRuleConfigurationBuilder.java  |  3 ++-
 .../config/YamlAuthorityRuleConfiguration.java |  7 +-
 .../AuthorityRuleConfigurationYamlSwapper.java |  8 ++-
 .../rule/builder/ShardingSphereRulesBuilder.java   |  6 ++---
 .../rule/builder/scope/GlobalRuleBuilder.java  |  5 +
 .../shardingsphere-infra-context/pom.xml   |  5 +
 .../context/metadata/MetaDataContextsBuilder.java  | 25 ++---
 .../metadata/MetaDataContextsBuilderTest.java  | 20 +
 .../GovernanceShardingSphereDataSource.java|  2 +-
 .../impl/AbstractBootstrapInitializer.java |  2 +-
 .../impl/GovernanceBootstrapInitializer.java   |  7 +++---
 .../impl/GovernanceBootstrapInitializerTest.java   |  1 -
 .../impl/StandardBootstrapInitializerTest.java | 26 ++
 .../src/test/resources/conf/local/server.yaml  | 10 ++---
 .../src/test/resources/conf/reg_center/server.yaml |  8 +++
 .../proxy/config/ProxyConfiguration.java   |  3 ---
 .../proxy/config/ProxyConfigurationLoader.java |  5 -
 .../config/yaml/YamlProxyServerConfiguration.java  |  2 --
 .../swapper/YamlProxyConfigurationSwapper.java |  6 ++---
 .../swapper/YamlProxyConfigurationSwapperTest.java | 23 +--
 .../src/test/resources/conf/server.yaml|  8 +--
 .../test/resources/docker/proxy/conf/server.yaml   |  8 +--
 .../adapter/ShardingSphereAdapterContainer.java| 16 -
 .../integration/engine/it/dql/GeneralDQLIT.java|  4 ++--
 .../resources/docker/db/proxy/conf/server.yaml |  8 +--
 .../proxy/conf/server.yaml |  8 +--
 .../proxy/conf/server.yaml |  8 +--
 .../docker/encrypt/proxy/conf/server.yaml  |  8 +--
 .../readwrite_splitting/proxy/conf/server.yaml |  8 +--
 .../sharding_governance/proxy/conf/server.yaml |  8 +--
 .../resources/docker/tbl/proxy/conf/server.yaml|  8 +--
 41 files changed, 236 insertions(+), 100 deletions(-)

diff --git a/examples/docker/sharding-proxy/sharding/conf/server.yaml 
b/examples/docker/sharding-proxy/sharding/conf/server.yaml
index 8544fab..43afc71 100644
--- a/examples/docker/sharding-proxy/sharding/conf/server.yaml
+++ b/examples/docker/sharding-proxy/sharding/conf/server.yaml
@@ -28,8 +28,12 @@
 #serverLists: localhost:2181
 #  overwrite: false
 
-users:
-  - root@%:root
+rules:
+  - !AUTHORITY
+users:
+  - root@%:root
+provider:
+  type: NATIVE
 
 props:
   max-connections-size-per-query: 1
diff --git 
a/examples/shardingsphere-proxy-example/shardingsphere-proxy-boot-mybatis-example/src/main/resources/conf/server.yaml
 
b/examples/shardingsphere-proxy-example/shardingsphere-proxy-boot-mybatis-example/src/main/resources/conf/server.yaml
index f07d391..7d9185c 100644
--- 
a/examples/shardingsphere-proxy-example/shardingsphere-proxy-boot-mybatis-example/src/main/resources/conf/server.yaml
+++ 
b/examples/shardingsphere-proxy-example/shardingsphere-proxy-boot-mybatis-example/src/main/resources/conf/server.yaml
@@ -33,9 +33,13 @@
 #  operationTimeoutMilliseconds: 500
 #  overwrite: false
 
-users:
-  - root@:root
-  - sharding@%:sharding
+rules:
+  - !AUTHORITY
+users:
+  - root@:root
+  - sharding@:sharding
+provider:
+  type: NATIVE
 
 props:
   max-connections-size-per-query: 1
diff --git 
a/examples/shardingsphere-proxy-example/shardingsphere-proxy-hint-example/src/main/resources/conf/server.yaml
 

[GitHub] [shardingsphere] tristaZero merged pull request #10287: Move the user configuration to authority rule

2021-05-10 Thread GitBox


tristaZero merged pull request #10287:
URL: https://github.com/apache/shardingsphere/pull/10287


   


-- 
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




[GitHub] [shardingsphere] tristaZero closed issue #10251: [API Change] Move the user configuration to authority rule

2021-05-10 Thread GitBox


tristaZero closed issue #10251:
URL: https://github.com/apache/shardingsphere/issues/10251


   


-- 
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




[GitHub] [shardingsphere] tristaZero commented on a change in pull request #10287: Move the user configuration to authority rule

2021-05-10 Thread GitBox


tristaZero commented on a change in pull request #10287:
URL: https://github.com/apache/shardingsphere/pull/10287#discussion_r629096519



##
File path: 
shardingsphere-infra/shardingsphere-infra-context/src/main/java/org/apache/shardingsphere/infra/context/metadata/MetaDataContextsBuilder.java
##
@@ -94,7 +93,7 @@ public StandardMetaDataContexts build() throws SQLException {
 for (String each : schemaRuleConfigs.keySet()) {
 mataDataMap.put(each, buildMetaData(each));
 }
-return new StandardMetaDataContexts(mataDataMap, 
buildGlobalSchemaMetaData(mataDataMap), executorEngine, new 
ShardingSphereUsers(users), props);
+return new StandardMetaDataContexts(mataDataMap, 
buildGlobalSchemaMetaData(mataDataMap), executorEngine, new 
ShardingSphereUsers(getUsersFromAuthorityRule()), props);

Review comment:
   `ShardingSphereUsers(users)` is expected to be removed from 
`StandardMetaDataContexts`. That way, `getUsersFromAuthorityRule ()` and 
`shardingsphere-infra-authority-api` of 
`shardingsphere-infra/shardingsphere-infra-context/pom.xml` can be deleted 
together. 
   
   Finally, all the content concerning `authority` and `users` will be 
restricted in the `Authority` global rule. That's our ideal result.
   
   
   @zhujunx This PR looks great really. I did not imagine we can merge it 
one! Thanks for your prompt nice work.
   @huanghao495430759 Please give a glance at this comment. Your issue has no 
block, go ahead pacing yourself.




-- 
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




[GitHub] [shardingsphere] terrymanu opened a new pull request #10294: Rename name config item form yaml for Readwrite-splitting

2021-05-10 Thread GitBox


terrymanu opened a new pull request #10294:
URL: https://github.com/apache/shardingsphere/pull/10294


   


-- 
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




[GitHub] [shardingsphere-elasticjob] skaic commented on issue #1889: IpUtils.getIp()方法获取不到ip

2021-05-10 Thread GitBox


skaic commented on issue #1889:
URL: 
https://github.com/apache/shardingsphere-elasticjob/issues/1889#issuecomment-836244835


   this is your problem


-- 
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