Skip to content

HIVE-29071: Add release workflow for HMS docker image #5899

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 7, 2025

Conversation

deniskuzZ
Copy link
Member

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@deniskuzZ
Copy link
Member Author

@ayushtkn, @simhadri-g, could you please help test it?

@deniskuzZ deniskuzZ changed the title Add docker image release workflow for HMS Add release workflow for HMS docker image Jun 25, 2025
@deniskuzZ
Copy link
Member Author

cc @zhangbutao, we'll probably need to include this into branch-4.1

@deniskuzZ deniskuzZ force-pushed the hms-docker-workflow branch 3 times, most recently from ce81754 to e17ee59 Compare June 28, 2025 21:04
@zhangbutao
Copy link
Contributor

I think this is a flaky test. https://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-5899/5/tests
Rerun the CI..

Error
expected:<[Transaction manager has aborted the transaction txnid:19.  Reason: Aborting [txnid:19,19] due to a write conflict on default/rebalance_test committed by [txnid:18,19] d/u]> but was:<[Lock acquisition for LockRequest(component:[LockComponent(type:SHARED_WRITE, level:TABLE, dbname:default, tablename:rebalance_test, operationType:DELETE, isTransactional:true, isDynamicPartitionWrite:false)], txnid:19, user:jenkins, hostname:hive-precommit-pr-5899-5-1mj6s-p9315-9qw7g, agentInfo:jenkins_20250701113933_84e365db-911f-4d1b-8da6-eb8929e3106d, zeroWaitReadEnabled:true, exclusiveCTAS:false) timed out after 6343466ms.  LockResponse(lockid:19, state:WAITING)]>
Stacktrace
org.junit.ComparisonFailure: expected:<[Transaction manager has aborted the transaction txnid:19.  Reason: Aborting [txnid:19,19] due to a write conflict on default/rebalance_test committed by [txnid:18,19] d/u]> but was:<[Lock acquisition for LockRequest(component:[LockComponent(type:SHARED_WRITE, level:TABLE, dbname:default, tablename:rebalance_test, operationType:DELETE, isTransactional:true, isDynamicPartitionWrite:false)], txnid:19, user:jenkins, hostname:hive-precommit-pr-5899-5-1mj6s-p9315-9qw7g, agentInfo:jenkins_20250701113933_84e365db-911f-4d1b-8da6-eb8929e3106d, zeroWaitReadEnabled:true, exclusiveCTAS:false) timed out after 6343466ms.  LockResponse(lockid:19, state:WAITING)]>
	at org.junit.Assert.assertEquals(Assert.java:117)
	at org.junit.Assert.assertEquals(Assert.java:146)
	at org.apache.hadoop.hive.ql.txn.compactor.TestCrudCompactorOnTez.testRebalanceCompactionWithParallelDeleteAsSecond(TestCrudCompactorOnTez.java:156)
	at org.apache.hadoop.hive.ql.txn.compactor.TestCrudCompactorOnTez.testRebalanceCompactionWithParallelDeleteAsSecondPessimisticLock(TestCrudCompactorOnTez.java:108)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.i

Copy link
Contributor

@zhangbutao zhangbutao left a comment

Choose a reason for hiding this comment

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

+1 LGTM

Rerun the CI because of flaky test

@deniskuzZ deniskuzZ marked this pull request as draft July 4, 2025 21:44
@deniskuzZ deniskuzZ force-pushed the hms-docker-workflow branch from 8d32c9e to bd09dc0 Compare July 6, 2025 08:53
@deniskuzZ deniskuzZ marked this pull request as ready for review July 6, 2025 08:53
@deniskuzZ deniskuzZ force-pushed the hms-docker-workflow branch 2 times, most recently from 39ab78e to 38f794f Compare July 6, 2025 08:55
@deniskuzZ deniskuzZ force-pushed the hms-docker-workflow branch from 38f794f to 97a4a38 Compare July 6, 2025 09:03
@deniskuzZ deniskuzZ changed the title Add release workflow for HMS docker image HIVE-29071: Add release workflow for HMS docker image Jul 6, 2025
Copy link

sonarqubecloud bot commented Jul 6, 2025

@deniskuzZ deniskuzZ merged commit 1a7fbd7 into apache:master Jul 7, 2025
6 checks passed
deniskuzZ added a commit to deniskuzZ/hive that referenced this pull request Jul 7, 2025
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