Hello Vihang Karajgaonkar,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/13259
to review the following change.
Change subject: Improve output for generate-schema-statements.py
......................................................................
Improve output for generate-schema-statements.py
The old output was a big wall of text that made it difficult to read for
developers.
This patch changes the output to use PrettyTable and output a table with
a row for each test table to be loaded or skipped. For example:
Target Dataset: tpcds
+----------------+---------------------------+--------+---------------------------------------------------------------------+
| DB | Table | Status | Notes
|
+----------------+---------------------------+--------+---------------------------------------------------------------------+
| tpcds | call_center | SKIP |
/test-warehouse/tpcds.call_center contains data |
| tpcds | catalog_page | SKIP |
/test-warehouse/tpcds.catalog_page contains data |
| tpcds | catalog_returns | SKIP |
/test-warehouse/tpcds.catalog_returns contains data |
| tpcds | catalog_sales | SKIP |
/test-warehouse/tpcds.catalog_sales contains data |
...
| tpcds | store_returns | SKIP |
/test-warehouse/tpcds.store_returns contains data |
| tpcds | store_sales | LOAD |
/test-warehouse/tpcds.store_sales does not exist or empty. |
...
| tpcds_seq_snap | store_sales_unpartitioned | SKIP | format not in included
list: ['text/none/none'] |
+----------------+---------------------------+--------+---------------------------------------------------------------------+
Change-Id: Idf53fde209bb7ee319a4c13667f3fa2783507072
---
M testdata/bin/generate-schema-statements.py
1 file changed, 73 insertions(+), 17 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/59/13259/1
--
To view, visit http://gerrit.cloudera.org:8080/13259
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Idf53fde209bb7ee319a4c13667f3fa2783507072
Gerrit-Change-Number: 13259
Gerrit-PatchSet: 1
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
