Uploaded image for project: 'Ignite'
  1. Ignite
  2. IGNITE-13136

Calcite integration. Improve join predicate testing.

Attach filesAttach ScreenshotAdd voteVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

    Details

      Description

      Currently we have to merge joining rows in order to test a join predicate:

          Row row = handler.concat(left, rightMaterialized.get(rightIdx++));
      
          if (!cond.test(row))
              continue;
      

      it results in unconditional building a joined row even if it will not be emitted to downstream further. To avoid extra GC pressure we need to test the join predicate before joining rows:

          if (!cond.test(left, right))
              continue;
      
          Row row = handler.concat(left, right);
      

        Attachments

        Issue Links

          Activity

            People

            • Assignee:
              zstan Stanilovsky Evgeny
              Reporter:
              rkondakov Roman Kondakov

              Dates

              • Created:
                Updated:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 3h 10m
                3h 10m

                  Issue deployment