azexcy commented on code in PR #23808:
URL: https://github.com/apache/shardingsphere/pull/23808#discussion_r1090195298
##########
kernel/data-pipeline/api/src/main/java/org/apache/shardingsphere/data/pipeline/api/check/consistency/DataConsistencyCheckResult.java:
##########
@@ -33,12 +31,33 @@ public final class DataConsistencyCheckResult {
private final DataConsistencyContentCheckResult contentCheckResult;
+ private final DataConsistencyCheckIgnoredType checkIgnoredType;
+
+ private final boolean ignored;
+
+ public DataConsistencyCheckResult(final DataConsistencyCountCheckResult
countCheckResult, final DataConsistencyContentCheckResult contentCheckResult) {
+ this.countCheckResult = countCheckResult;
+ this.contentCheckResult = contentCheckResult;
+ checkIgnoredType = null;
+ ignored = false;
+ }
+
+ public DataConsistencyCheckResult(final DataConsistencyCheckIgnoredType
checkIgnoredType) {
+ this.checkIgnoredType = checkIgnoredType;
+ ignored = true;
+ countCheckResult = null;
+ contentCheckResult = null;
+ }
+
/**
* Is count and content matched.
*
* @return matched or not
*/
public boolean isMatched() {
+ if (ignored || null == countCheckResult || null == contentCheckResult)
{
Review Comment:
Ok, add it just because idea warning, removed
--
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]
