Github user fhueske commented on the pull request:
https://github.com/apache/flink/pull/1477#issuecomment-168736055
Hi @gallenvara, thanks for this PR!
Explain is a very nice feature for the Table API (and later on for the SQL
interface).
IMO, it would be better to reuse the existing JSON plan and generate the
HIVE visualization locally inside of the Table API. That would avoid to
replicate quite a bit of functionality just for the sake of a different
visualization. Would that work as well?
In general, it is a good idea to discuss changes that modify user-facing
APIs and touch several components/modules before starting to code as pointed
out in our [contribution
guidelines](http://flink.apache.org/contribute-code.html).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
