Skip to content

MAHOUT-865: Add release.md #866

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.

Sign up for GitHub

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

Jump to bottom
Merged
guan404ming merged 1 commit into from
Jan 19, 2026
Merged

MAHOUT-865: Add release.md #866

guan404ming merged 1 commit into from
Jan 19, 2026

Conversation

@guan404ming
Copy link
Member

@guan404ming guan404ming commented Jan 18, 2026 •
edited
Loading

Purpose of PR

Add detailed release step documentation

ref:

Related Issues or PRs

Changes Made

  • Bug fix
  • New feature
  • Refactoring
  • Documentation
  • Test
  • CI/CD pipeline
  • Other

Breaking Changes

  • Yes
  • No

Checklist

  • Added or updated unit tests for all changes
  • Added or updated documentation for all changes
  • Successfully built and ran all unit tests or manual tests locally
  • PR title follows "MAHOUT-XXX: Brief Description" format (if related to an issue)
  • Code follows ASF guidelines

@guan404ming
Copy link
Member Author

guan404ming commented Jan 18, 2026

cc @400Ping

viiccwen
viiccwen reviewed Jan 19, 2026
View reviewed changes
Copy link
Contributor

@viiccwen viiccwen left a comment

Choose a reason for hiding this comment

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

Hello, Is it template or written by yourself?

@guan404ming
Copy link
Member Author

guan404ming commented Jan 19, 2026 •
edited
Loading

Hello, Is it template or written by yourself?

This is written by myself but with Claude Code assistance. Highly refers to airflow release doc.

@viiccwen
Copy link
Contributor

viiccwen commented Jan 19, 2026

This is written by myself but with Claude Code assistance. Highly refers to airflow release doc.

got it!

400Ping
400Ping approved these changes Jan 19, 2026
View reviewed changes
Copy link
Contributor

@400Ping 400Ping left a comment

Choose a reason for hiding this comment

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

Overall LGTM

ryankert01
ryankert01 approved these changes Jan 19, 2026
View reviewed changes
Copy link
Member

@ryankert01 ryankert01 left a comment

Choose a reason for hiding this comment

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

lg, btw, will we have nightly build in the future?

@rawkintrevo
Copy link
Contributor

rawkintrevo commented Jan 19, 2026 •
edited
Loading

The doc references qumat and qumat-qdp as if they're separate things. Do we want that?

Edit:

@ryankert01 we've had nightly builds in the past (they were published with Jenkins to let you know how long ago in the past). I'm down for doing them again, but lets get releases done first. Also, nightlies are not considered releases, for bureaucratic reasons.

rich7420
rich7420 approved these changes Jan 19, 2026
View reviewed changes
Copy link
Contributor

@rich7420 rich7420 left a comment

Choose a reason for hiding this comment

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

@guan404ming thanks for the patch!

@rawkintrevo
Copy link
Contributor

rawkintrevo commented Jan 19, 2026

This LGTM- but be prepared to change as we learn the new process with Apache Trusted Releases (ATR)

@guan404ming
Copy link
Member Author

guan404ming commented Jan 19, 2026 •
edited
Loading

The doc references qumat and qumat-qdp as if they're separate things. Do we want that?

Yes, this is intended. the qumat-qdp serve as a optional extra for qumat just like the airflow-core and the airflow-provider, which let user could choose to pip install qumat (only for qumat-core) or pip install qumat[qdp] (to include the qdp)

ref: https://pypi.org/project/apache-airflow/ and https://pypi.org/project/apache-airflow-providers-celery/

@guan404ming guan404ming mentioned this pull request Jan 19, 2026
Merged
14 tasks
@rawkintrevo
Copy link
Contributor

rawkintrevo commented Jan 19, 2026 •
edited
Loading

Why not just make it an extra in the pyproject.toml?

pip install qumat[qdp] ?

Edit: The thread is split across PRs (my fault), let's continue discussion on #875

@guan404ming
Copy link
Member Author

guan404ming commented Jan 19, 2026

Feel free to open pr to refine this, thanks!

@guan404ming guan404ming merged commit 6762f85 into apache:main Jan 19, 2026
2 checks passed
@guan404ming guan404ming deleted the add-release-md branch January 19, 2026 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ryankert01 ryankert01 ryankert01 approved these changes

@rich7420 rich7420 rich7420 approved these changes

@andrewmusselman andrewmusselman Awaiting requested review from andrewmusselman

@rawkintrevo rawkintrevo Awaiting requested review from rawkintrevo

+2 more reviewers

@viiccwen viiccwen viiccwen left review comments

@400Ping 400Ping 400Ping approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Docs] Add Release.md

6 participants

@guan404ming @viiccwen @rawkintrevo @400Ping @ryankert01 @rich7420