Skip to content

Comet sink operator should not have children operators #25

Closed

Description

@viirya

Describe the bug

This fixes incorrect native operator when transforming Spark query plan to Comet query plan. We now add children operators into native operator anyway in operator2Proto. But for Comet sink operators e.g. shuffle, they don't have children operators but work as a input source. This doesn't cause issue because we don't actually plan these children for sink operators in native planner, but the native operator (proto message) looks incorrect and confusing.

Steps to reproduce

No response

Expected behavior

No response

Additional context

No response

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions