Skip to content

[WIP]HIVE-25542: Remove references to Index Configurations - #6815

Open
maheshrajus wants to merge 1 commit into
apache:masterfrom
maheshrajus:HIVE-25542
Open

maheshrajus wants to merge 1 commit into
apache:masterfrom
maheshrajus:HIVE-25542

Conversation

@maheshrajus

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Removes the obsolete hive.optimize.index.filter configuration and every reference to it.

Why are the changes needed?

Hive indexes were removed in 4.0.0 (HIVE-18715), but hive.optimize.index.filter survived as dead configuration. The
name is actively misleading — it no longer has anything to do with indexes; it gates ordinary predicate push-down
to the table scan, which hive.optimize.ppd already controls. Keeping it means two knobs for one behavior, a
confusing entry in the config docs, and hundreds of set lines in qtests that suggest a toggle users should care
about.

Does this PR introduce any user-facing change?

Yes, two:

  1. The config(hive.optimize.index.filter) is gone. Because hive.optimize.index.filter defaulted to true, anyone leaving it at the default sees
    no behavior change. Anyone explicitly setting it to false to suppress predicate push-down loses that control and
    must use hive.optimize.ppd=false instead. With hive.conf.validation=true (the default), setting the removed key now
    raises an unknown-config error rather than being silently ignored.
  2. Error message text changed in LeafFilterFactory for unsupported Parquet FilterPredicate conversions — it now
    recommends only set hive.optimize.ppd=false.

How was this patch tested?

Existing tests

@sonarqubecloud

Copy link
Copy Markdown

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants