[DRILL-5405] Add missing operator types without dependency on protobuf enum

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.10.0
• Fix Version/s: 1.19.0
• Assignee: Vova Vysotskyi
• Reporter: Arina Ielchiieva
• Reviewer: Karthikeyan Manivannan
• Created: 31/Mar/17 08:45
• Updated: 12/Jan/21 23:07
• Resolved: 12/Jan/21 23:07

Description

Source - https://github.com/apache/drill/pull/214#discussion_r42687575

When operators are not added into CoreOperatorType enum, they are displayed on Web UI as UNKNOWN_OPERATOR.
Screenshots:
now -> unknown_operator.JPG
should be -> maprdb_sub_scan.JPG

Though plugins work fine without this changes, it's still would be nice to display operator type on Web UI.
But dependency on protobuf enum removes the ability for format plugins to truly be drop-in pluggable.
In this case as option we may need to refactor this interface to just return a String and not use the protobuf enum, or have a separate "notes" field if we want to have a generic filescan operator ID in this enum with extra data to say what kind of scan we used.

Issue Links

• is related to: DRILL-6643 OperatorType for Values and IteratorValidator
• links to: GitHub Pull Request #804