Skip to content

Use the HiveTableHandle enforced constraint when loading stats#13488

Merged
findepi merged 1 commit intotrinodb:masterfrom
alexjo2144:hive/collect-filter-statistics
Aug 4, 2022
Merged

Use the HiveTableHandle enforced constraint when loading stats#13488
findepi merged 1 commit intotrinodb:masterfrom
alexjo2144:hive/collect-filter-statistics

Conversation

@alexjo2144
Copy link
Member

Description

Load partitions selectively based on the TableHandle's enforced constraint

Is this change a fix, improvement, new feature, refactoring, or other?

Fix

Is this a change to the core query engine, a connector, client library, or the SPI interfaces? (be specific)

Hive Connector

How would you describe this change to a non-technical end user or system administrator?

Reduce loading time for Hive partitions by using the query filter

Related issues, pull requests, and links

Introduced by: #11877

Documentation

(x) No documentation is needed.
( ) Sufficient documentation is included in this PR.
( ) Documentation PR is available with #prnumber.
( ) Documentation issue #issuenumber is filed, and can be handled later.

Release notes

(x) No release notes entries required.
( ) Release notes entries required with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

@cla-bot cla-bot bot added the cla-signed label Aug 3, 2022
@alexjo2144 alexjo2144 force-pushed the hive/collect-filter-statistics branch 2 times, most recently from f0e6a46 to 3251ac3 Compare August 3, 2022 22:30
@alexjo2144 alexjo2144 force-pushed the hive/collect-filter-statistics branch from 3251ac3 to b99b181 Compare August 4, 2022 02:02
@findepi findepi merged commit a10aa9b into trinodb:master Aug 4, 2022
@findepi
Copy link
Member

findepi commented Aug 4, 2022

@alexjo2144 can you please suggest RN wording under #13474 ?

@alexjo2144 alexjo2144 deleted the hive/collect-filter-statistics branch August 4, 2022 15:41
@github-actions github-actions bot added this to the 393 milestone Aug 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants