Add output types back to EXPLAIN output #382
Conversation
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@prestosql.io. If you are contributing on behalf of someone else (e.g., your employer), the individual CLA may not be sufficient and your employer may need the Corporate CLA signed. |
| @@ -137,4 +137,26 @@ public String getDetails() | |||
| { | |||
| return estimatedCost; | |||
| } | |||
|
|
|||
| public static class OutputSymbol | |||
There was a problem hiding this comment.
TypedSymbol, SymbolAndType? "Output" is a property of how it's used, but the concept this represents is a symbol + its type.
There was a problem hiding this comment.
Addressed as separate commit, I do not feel comfortable editing (commit) code that is authored by some else. Should I squash it?
There was a problem hiding this comment.
Yes, please squash it. That is ok, it’s similar to applying comments for someone else’s change before merging a PR.
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@prestosql.io. If you are contributing on behalf of someone else (e.g., your employer), the individual CLA may not be sufficient and your employer may need the Corporate CLA signed. |
Output types were removed unintentionally in 1630408 Extracted from: prestodb/presto#12393
kokosing
force-pushed
the
origin/master/101_port_12393_from_legacy_presto
branch
from
4b48864
to
67c2e63
Compare
|
Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please submit the signed CLA to cla@prestosql.io. If you are contributing on behalf of someone else (e.g., your employer), the individual CLA may not be sufficient and your employer may need the Corporate CLA signed. |
Add output types back to EXPLAIN output
Output types were removed unintentionally in
1630408
Extracted from: prestodb/presto#12393