korlov42 commented on a change in pull request #9009:
URL: https://github.com/apache/ignite/pull/9009#discussion_r617445038
##########
File path:
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/trait/DistributionTrait.java
##########
@@ -142,7 +142,7 @@
&& DistributionFunction.satisfy(function, other.function));
if (other.getType() == RANDOM_DISTRIBUTED)
- return getType() == HASH_DISTRIBUTED;
+ return getType() == HASH_DISTRIBUTED || getType() == SINGLETON;
Review comment:
I don't sure this is a legit change. If SINGLETON will satisfy a RANDOM
distribution, then there will be no additional exchange, which leads to
singleton be a part of fragment with RANDOM distribution, but this should not
be possible, since this fragment is allowed to be executed on several nodes,
whereas SINGLETON is expected to be executed on only one
--
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:
[email protected]
