You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2022/06/26 04:14:27 UTC

[GitHub] [shardingsphere] git000 opened a new pull request, #18602: [Issue#18446] Add test case for AlterTransactionRuleHandler

git000 opened a new pull request, #18602:
URL: https://github.com/apache/shardingsphere/pull/18602

   Fixes #18446 .


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org

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

[GitHub] [shardingsphere] git000 commented on pull request #18602: [Issue#18446] Add test case for AlterTransactionRuleHandler

Posted by GitBox <gi...@apache.org>.
git000 commented on PR #18602:
URL: https://github.com/apache/shardingsphere/pull/18602#issuecomment-1166434016

   Error on jdk11:
   ShowTransactionRuleHandlerTest.assertExecutorWithLocal:71 » NullPointer
   ShowTransactionRuleHandlerTest.assertExecutorWithXA:52 » NullPointer
   
    java.lang.NullPointerException
   2022-06-26T05:29:39.4540896Z 	at org.apache.shardingsphere.proxy.backend.text.distsql.ral.common.queryable.ShowTransactionRuleHandler.getRows(ShowTransactionRuleHandler.java:52)
   2022-06-26T05:29:39.4542025Z 	at org.apache.shardingsphere.proxy.backend.text.distsql.ral.QueryableRALBackendHandler.createMergedResult(QueryableRALBackendHandler.java:63)
   2022-06-26T05:29:39.4543055Z 	at org.apache.shardingsphere.proxy.backend.text.distsql.ral.QueryableRALBackendHandler.execute(QueryableRALBackendHandler.java:54)
   2022-06-26T05:29:39.4544257Z 	at org.apache.shardingsphere.proxy.backend.text.distsql.ral.common.queryable.ShowTransactionRuleHandlerTest.assertExecutorWithLocal(ShowTransactionRuleHandlerTest.java:71)
   
   execute ShowTransactionRuleHandlerTest on the local environment without error
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org

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

[GitHub] [shardingsphere] terrymanu closed pull request #18602: [Issue#18446] Add test case for AlterTransactionRuleHandler

Posted by GitBox <gi...@apache.org>.
terrymanu closed pull request #18602: [Issue#18446] Add test case for AlterTransactionRuleHandler
URL: https://github.com/apache/shardingsphere/pull/18602


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org

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

[GitHub] [shardingsphere] terrymanu commented on pull request #18602: [Issue#18446] Add test case for AlterTransactionRuleHandler

Posted by GitBox <gi...@apache.org>.
terrymanu commented on PR #18602:
URL: https://github.com/apache/shardingsphere/pull/18602#issuecomment-1167118226

   The issue is merged by other PR, I just close 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.

To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org

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

[GitHub] [shardingsphere] git000 commented on pull request #18602: [Issue#18446] Add test case for AlterTransactionRuleHandler

Posted by GitBox <gi...@apache.org>.
git000 commented on PR #18602:
URL: https://github.com/apache/shardingsphere/pull/18602#issuecomment-1166447134

   > Please fix the CI
   
   Error on jdk11:
   ShowTransactionRuleHandlerTest.assertExecutorWithLocal:71 » NullPointer
   ShowTransactionRuleHandlerTest.assertExecutorWithXA:52 » NullPointer
   
   java.lang.NullPointerException
   2022-06-26T05:29:39.4540896Z at org.apache.shardingsphere.proxy.backend.text.distsql.ral.common.queryable.ShowTransactionRuleHandler.getRows(ShowTransactionRuleHandler.java:52)
   2022-06-26T05:29:39.4542025Z at org.apache.shardingsphere.proxy.backend.text.distsql.ral.QueryableRALBackendHandler.createMergedResult(QueryableRALBackendHandler.java:63)
   2022-06-26T05:29:39.4543055Z at org.apache.shardingsphere.proxy.backend.text.distsql.ral.QueryableRALBackendHandler.execute(QueryableRALBackendHandler.java:54)
   2022-06-26T05:29:39.4544257Z at org.apache.shardingsphere.proxy.backend.text.distsql.ral.common.queryable.ShowTransactionRuleHandlerTest.assertExecutorWithLocal(ShowTransactionRuleHandlerTest.java:71)
   
   execute ShowTransactionRuleHandlerTest on the local environment without error
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@shardingsphere.apache.org

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