Github user jinfengni commented on the issue:
https://github.com/apache/drill/pull/436
I'll will regression and merge this PR.
---
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
Github user jinfengni commented on the issue:
https://github.com/apache/drill/pull/436
LGTM.
+1
Thanks for the PR, @arina-ielchiieva !
---
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
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/436
@jinfengni, agree. Added schema in the output and updated PR comment with
example.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user jinfengni commented on the issue:
https://github.com/apache/drill/pull/436
The new output looks better.
I checked hive's command. Seems it lists schema name
and its relevant properties. Does it make sense to add schema | database name
in the output?
---
If
Github user arina-ielchiieva commented on the issue:
https://github.com/apache/drill/pull/436
@jinfengni
PR is updated, physical_location is removed.
---
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