Also check for Presto Native operator names in QueryStats.

[spershin]

Presto Native operator names are different.

Description

To properly report 'shuffled' and 'raw input' query stats when running Presto Native, we also check Presto Native operator names.

== NO RELEASE NOTE ==

[xiaoxmeng]

@spershin thanks!

[tdcmeehan]

Wondering if we can just add a boolean flag on the OperatorStats indicating that this was for an exchange? This actually would remove the need to hardcode operator names or check if we're using native execution altogether. I took a brief look and it looks like, at least from the Java side, it should be pretty straightforward.

[xiaoxmeng]

@spershin Thanks for the update!

[spershin]

@tdcmeehan

So, does this mean that we want two bool flags?
One for Exchange/Merge and another one for Scan?

[tdcmeehan]

@spershin yes, that works.

[spershin]

@tdcmeehan

I might need some pointer to code site(s) where we populate the OperatorStats.
Members like operatorType, operatorId, planNodeId and such.
It is not trivial to find for some reason.

[spershin]

NVM, found it.

[spershin]

I don't think it is a good idea to add two struct fields just for that.
Closing the PR.