hive icon indicating copy to clipboard operation
hive copied to clipboard

HIVE-29259: Remove label from text based output for desc catalog

Open Aggarwal-Raghav opened this issue 1 month ago • 3 comments

What changes were proposed in this pull request?

Please check the discussion in HIVE-29259

Why are the changes needed?

To have to correct output format for desc catalog as its there for desc database

Does this PR introduce any user-facing change?

Yes, User will observe different table output format Older:

0: jdbc:hive2://127.0.0.1:10000/default> desc catalog hive;
+------------------+---------------------------------------------------------+------------+
|   cat_name     |                      comment              | location  |
+------------------+-------------------------------------------------------- +-------------+
| Catalog Name    | hive                                    | NULL    |
| Comment       | Default catalog for Hive                       | NULL    |
| Location      | hdfs://127.0.0.1:8028/user/hive/warehouse/hiveicetest   | NULL     |
+------------------+---------------------------------------------------------+------------+

New:

0: jdbc:hive2://127.0.0.1:10000/default> desc catalog hive;
+------------------+-------------------------------------------------------+-------------------------------------------------------------+
|   cat_name     |                 comment                   |              location                                      |
+------------------+-------------------------------------------------------+-------------------------------------------------------------+
|      hive       |       Default catalog for Hive              | hdfs://127.0.0.1:8028/user/hive/warehouse/hiveicetest      |
+------------------+-------------------------------------------------------+-------------------------------------------------------------+

How was this patch tested?

On local Setup and ran catalog.q

Aggarwal-Raghav avatar Dec 08 '25 19:12 Aggarwal-Raghav

After fix when hive.ddl.output.format = text: Screenshot 2025-12-09 at 12 07 01 AM

After fix when hive.ddl.output.format = json: Screenshot 2025-12-09 at 12 08 18 AM

Aggarwal-Raghav avatar Dec 08 '25 19:12 Aggarwal-Raghav

CC @zhangbutao

Aggarwal-Raghav avatar Dec 08 '25 19:12 Aggarwal-Raghav

I merged this PR since it already had two approvals. Thanks for the PR @Aggarwal-Raghav and @deniskuzZ @zhangbutao for the reviews!

zabetak avatar Dec 16 '25 12:12 zabetak