comecny commented on code in PR #32776:
URL: https://github.com/apache/shardingsphere/pull/32776#discussion_r1742276724
##########
features/sharding/core/src/main/java/org/apache/shardingsphere/sharding/merge/dal/show/ShowTableStatusMergedResult.java:
##########
@@ -74,10 +74,16 @@ private void merge(final MemoryQueryResultRow row, final
MemoryQueryResultRow ne
}
private BigInteger sum(final Object num1, final Object num2) {
+ if (num1 == null || num2 == null){
+ return BigInteger.ZERO;
Review Comment:
if the bug is caused by the view , then is wouldn't be a case where on is
null and then other is not . but ,it is indeed not elegant ! changed.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
