[HIVE-26401] Refine the log of add_partitions if the partition already exists

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 4.0.0-alpha-1
• Component/s: Hive
• Labels: pull-request-available
• Target Version/s: 4.0.0-alpha-2
• Assignee: Wechar
• Reporter: Wechar
• Created: 16/Jul/22 11:27
• Updated: 21/Oct/22 07:20

Description

Currently add_partitions_xxx will log the complete information of a partition if it already exists, see in HMSHandler.java#L4320:

    if (!shouldAdd) {
      LOG.info("Not adding partition {} as it already exists", part);
      return false;
    }

It will print a long message including the columns of this partition, we think it is unnecessary based on the following two points:
1. The long message is redundant.
We can get enough information from just cat_name.db_name.tbl_name[part_col1=part_val1/part_col2=part_val2...]

2. The long message is not friendly to save and query.
This log message will take up a large log space especially when the user need to execute MSCK REPAIR TABLE operation regularly because the old partition must be already existed. 

Issue Links

• links to: GitHub Pull Request #3447