voonhous commented on code in PR #19959:
URL: https://github.com/apache/hudi/pull/19959#discussion_r4018217799
##########
release/release_guide.md:
##########
@@ -312,7 +312,25 @@ Source Release step) -- otherwise the voted tarball ships
a `-SNAPSHOT` Trino pi
5. Verify the released Trino resolves from Central against an empty local
repository
(scope the check to io.trino: the module's hudi siblings are not on Central
until this release completes):
`mvn dependency:get -Dartifact=io.trino:trino-hive:NNN
-Dmaven.repo.local=$(mktemp -d)`
-6. CI and the E2E workflow then run with zero SPI drift; the staging deploy
flow in "Build a release candidate"
+6. Check for dependency drift before cutting the RC: hudi-trino compiles and
tests against Hudi's managed versions,
+ while the plugin bundles Trino's `NNN` versions. Dispatch the drift check
on the release branch and read the
+ table in the run's job summary (exit 1 and a `Dependency drift` warning
mean drift):
Review Comment:
Fixed in 1bff58b371a3: the step now says the dispatched run stays green and
reports drift through the warning, summary table and issue; exit 1 is only for
the local script run.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
