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 2021/11/29 08:35:07 UTC

[GitHub] [shardingsphere] TeslaCN opened a new issue #13848: Consider replacing java.util.stream with for each in high frequency codes

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


   ## Refactor
   
   ## Testing
   
   Using BenchmarkSQL (adapts ShardingSphere JDBC) connects to single node openGauss.
   ShardingSphere JDBC sharding data to 1 shards, which means the data was not sharded.
   
   Terminals: 696 
   Runs for 5 min.
   
   ## Measured tpmC
   
   ### Connects to openGauss directly
   ![image](https://user-images.githubusercontent.com/20503072/143833791-4a5d7d47-eaf4-4207-b994-5a33050db8b8.png)
   
   ### ShardingSphere JDBC master branch
   commit 93f116da950ea0173b1c8d9faec20d9f91e04846
   ![image](https://user-images.githubusercontent.com/20503072/143833814-6bb70be5-f73f-4c9f-8b8c-80361268ada5.png)
   
   ### ShardingSphere JDBC reduced stream
   Refer to #13845 
   ![image](https://user-images.githubusercontent.com/20503072/143833837-f3da3527-60be-46fe-9f51-fcbd4b3c9f03.png)
   
   
   ## Profiling
   
   ### Master branch
   commit 93f116da950ea0173b1c8d9faec20d9f91e04846
   
   #### Samples of `org.apache.shardingsphere`
   ![image](https://user-images.githubusercontent.com/20503072/143833010-02634a80-3eef-4b92-ba57-11d57923647d.png)
   
   #### Samples of `java.util.stream`
   ![image](https://user-images.githubusercontent.com/20503072/143833136-a8cc149d-bd60-4f99-b81f-f7b0e8d2623e.png)
   
   ### Replaced stream with for each
   
   #### Samples of `org.apache.shardingsphere`
   
   ![image](https://user-images.githubusercontent.com/20503072/143833174-e2993464-49c7-4d44-a5b5-0623b8fd8ad3.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.

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

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

[GitHub] [shardingsphere] TeslaCN commented on issue #13848: Consider replacing java.util.stream with for each in high frequency codes

Posted by GitBox <gi...@apache.org>.

TeslaCN commented on issue #13848:
URL: https://github.com/apache/shardingsphere/issues/13848#issuecomment-1081346150


   We have removed most of them.


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

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

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

[GitHub] [shardingsphere] TeslaCN closed issue #13848: Consider replacing java.util.stream with for each in high frequency codes

Posted by GitBox <gi...@apache.org>.

TeslaCN closed issue #13848:
URL: https://github.com/apache/shardingsphere/issues/13848


   


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