ygerzhedovich commented on a change in pull request #411:
URL: https://github.com/apache/ignite-3/pull/411#discussion_r740255806
##########
File path:
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/prepare/QueryTemplate.java
##########
@@ -104,7 +103,7 @@ public ExecutionPlan map(PlanningContext ctx) {
private List<Fragment> replace(List<Fragment> fragments, Fragment
fragment, List<Fragment> replacement) {
assert !nullOrEmpty(replacement);
- Map<Long, Long> newTargets = new HashMap<>();
+ Long2LongOpenHashMap newTargets = new Long2LongOpenHashMap();
Review comment:
we must revise all usages of new intMap, for example, few lines below we
get Long and check in on null - it's wrong way.
--
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]
