Hello Vihang Karajgaonkar, Zoltan Borok-Nagy, Tim Armstrong, Joe McDonnell, 
Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/13259

to look at the new patch set (#2).

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, 74 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/59/13259/2
--
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: newpatchset
Gerrit-Change-Id: Idf53fde209bb7ee319a4c13667f3fa2783507072
Gerrit-Change-Number: 13259
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com>
Gerrit-Reviewer: Zoltan Borok-Nagy <borokna...@cloudera.com>

Reply via email to