Skip to content

HIVE-23832: Compaction cleaner fails to clean up deltas when using bl…#1243

Merged
deniskuzZ merged 1 commit intoapache:masterfrom
deniskuzZ:HIVE-23832
Jul 14, 2020
Merged

HIVE-23832: Compaction cleaner fails to clean up deltas when using bl…#1243
deniskuzZ merged 1 commit intoapache:masterfrom
deniskuzZ:HIVE-23832

Conversation

@deniskuzZ
Copy link
Member

…ocking compaction

NOTICE

Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HIVE-XXXXX: Fix a typo in YYY)
For more details, please see https://cwiki.apache.org/confluence/display/Hive/HowToContribute

//double wait time until 5min
waitTimeMs = waitTimeMs*2;
waitTimeMs = Math.max(waitTimeMs, waitTimeOut);
waitTimeMs = Math.min(waitTimeMs, waitTimeOut);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need this change? Per the documentation, the minimum wait timeout is 2000 ms

Copy link
Member Author

@deniskuzZ deniskuzZ Jul 13, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hive.compactor.wait.timeout default is 300000ms (5 min) - that's max,
start wait time = (waitTimeMs = 1000) X 2 = 2000ms;

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've got your point.

@deniskuzZ deniskuzZ merged commit ac8cc9b into apache:master Jul 14, 2020
abstractdog pushed a commit to abstractdog/hive that referenced this pull request Apr 9, 2021
…en using blocking compaction (Denys Kuzmenko, reviewed by Laszlo Pinter)

Closes (apache#1243)

Change-Id: I42036fb5cc504132e50e25b89e9aa31d92c37024
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.

3 participants