Skip to content

Conversation

@blrchen
Copy link
Collaborator

@blrchen blrchen commented Nov 1, 2022

Description

This PR adds release instructions for Release Candidate. The updated doc can be previewed at https://github.com/feathr-ai/feathr/blob/blair/rc-release-doc/docs/dev_guide/feathr_overall_release_guide.md

How was this PR tested?

Does this PR introduce any user-facing changes?

  • No. You can skip the rest of this section.
  • Yes. Make sure to clarify your proposed changes.

xiaoyongzhu
xiaoyongzhu previously approved these changes Nov 1, 2022
Yuqing-cat
Yuqing-cat previously approved these changes Nov 1, 2022
@blrchen blrchen dismissed stale reviews from Yuqing-cat and xiaoyongzhu via dfee705 November 1, 2022 10:26
Copy link
Collaborator

@mufajjul mufajjul left a comment

Choose a reason for hiding this comment

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

All looks good.

@blrchen blrchen merged commit 02e3643 into main Nov 1, 2022
@blrchen blrchen deleted the blair/rc-release-doc branch November 1, 2022 11:19
loomlike pushed a commit to loomlike/feathr that referenced this pull request Nov 1, 2022
* Add release instructions for Release Candidate

* Add a section for release versioning

* Add a section for overall process triggered by the release manager
xiaoyongzhu pushed a commit that referenced this pull request Nov 23, 2022
* Fix xdist test error. Also make a small cleanup some codes

Signed-off-by: Jun Ki Min <[email protected]>

* Revert "Revert 756 (#798)"

This reverts commit ff438f5.

* revert 798 (revert756 - example notebook refactor). Also add job_utils unit tests

Signed-off-by: Jun Ki Min <[email protected]>

* Update test_azure_spark_e2e.py

* Fix doc dead links (#805)

This PR fixes dead links detected in latest ci run. The doc scan ci action has been updated to run on main only, as running this in PR frequently reports false alarm due to changes in CI not deployed.

* Improve UI experience and clean up ui code warnings (#801)

* Add DataSourcesSelect and FlowGraph and ResizeTable components. Fix all warning and lint issues.

Signed-off-by: Boli Guan <[email protected]>

* Add CardDescriptions component and fix ESlint warning.

Signed-off-by: Boli Guan <[email protected]>

* Update FeatureDetails page title.

Signed-off-by: Boli Guan <[email protected]>

* Rename ProjectSelect

Signed-off-by: Boli Guan <[email protected]>

Signed-off-by: Boli Guan <[email protected]>

* Add release instructions for Release Candidate (#809)

* Add release instructions for Release Candidate

* Add a section for release versioning

* Add a section for overall process triggered by the release manager

* Bump version to 0.9.0-rc1 (#810)

* Fix tests to use mocks and fix get_result_df's databricks behavior

Signed-off-by: Jun Ki Min <[email protected]>

* fix tem file to dir

Signed-off-by: Jun Ki Min <[email protected]>

* checkout the feature_derivations.py from main (it was temporally changed to goaround previous issues)

Signed-off-by: Jun Ki Min <[email protected]>

* Remove old databricks sample notebook. Change pip install feathr from the github main branch to pickup the latest changes always

Signed-off-by: Jun Ki Min <[email protected]>

* Fix config and get_result_df for synapse

* Fix generate_config to accept all the feathr env var config name

Signed-off-by: Jun Ki Min <[email protected]>

* Add more pytests

Signed-off-by: Jun Ki Min <[email protected]>

* Use None as default dataformat in the job_utils. Instead, set 'avro' as a default output format to the job tags from the client

Signed-off-by: Jun Ki Min <[email protected]>

* Change feathr client to mocked object

Signed-off-by: Jun Ki Min <[email protected]>

* Change timeout to 1000s in the notebook

Signed-off-by: Jun Ki Min <[email protected]>

Signed-off-by: Jun Ki Min <[email protected]>
Signed-off-by: Boli Guan <[email protected]>
Co-authored-by: Blair Chen <[email protected]>
Co-authored-by: Blair Chen <[email protected]>
Co-authored-by: Boli Guan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants