Skip to content
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

[featurn] support flyway database migration #1875

Merged
merged 14 commits into from
Apr 28, 2024
Merged

[featurn] support flyway database migration #1875

merged 14 commits into from
Apr 28, 2024

Conversation

tomsun28
Copy link
Contributor

What's changed?

support flyway database migration

Checklist

  • I have read the Contributing Guide
  • I have written the necessary doc or comment.
  • I have added the necessary unit tests and all cases have passed.

Add or update API

  • I have added the necessary e2e tests and all cases have passed.

Signed-off-by: tomsun28 <tomsun28@outlook.com>
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

license-eye has checked 2205 files.

Valid Invalid Ignored Fixed
881 3 1321 0
Click to see the invalid file list
  • manager/src/main/resources/db/migration/h2/V160__update_column.sql
  • manager/src/main/resources/db/migration/mysql/V160__update_column.sql
  • manager/src/main/resources/db/migration/postgresql/V160__update_column.sql
Use this command to fix any missing license headers
```bash

docker run -it --rm -v $(pwd):/github/workspace apache/skywalking-eyes header fix

</details>

tomsun28 and others added 9 commits April 28, 2024 09:42
Signed-off-by: tomsun28 <tomsun28@outlook.com>
….sql

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: tomsun28 <tomsun28@outlook.com>
…e_column.sql

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: tomsun28 <tomsun28@outlook.com>
…umn.sql

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Signed-off-by: tomsun28 <tomsun28@outlook.com>
Signed-off-by: tomsun28 <tomsun28@outlook.com>
Signed-off-by: tomsun28 <tomsun28@outlook.com>
Signed-off-by: tomsun28 <tomsun28@outlook.com>
@tomsun28 tomsun28 marked this pull request as ready for review April 28, 2024 07:05
@github-actions github-actions bot added the doc Improvements or additions to documentation label Apr 28, 2024
@tomsun28 tomsun28 added this to the 1.6.0 milestone Apr 28, 2024
@tomsun28
Copy link
Contributor Author

fix #1801

@tomsun28
Copy link
Contributor Author

-- for to hertzbeat v1.6.0 upgrade in mysql, user need run this by themself, ignore for non-upgrading -> 1.6.0 users

# UPDATE hzb_param SET param_value = `value` WHERE `value` IS NOT NULL AND param_value IS NULL;
# ALTER TABLE hzb_param DROP COLUMN `value`;
# commit;
# 
# UPDATE hzb_tag SET tag_value = `value` WHERE `value` IS NOT NULL AND tag_value IS NULL;
# ALTER TABLE hzb_tag DROP COLUMN `value`;
# commit;
# 
# UPDATE hzb_status_page_history SET unknowing = `unknown` WHERE `unknown` IS NOT NULL AND unknowing IS NULL;
# ALTER TABLE hzb_status_page_history DROP COLUMN `unknown`;
# commit;

@tomsun28
Copy link
Contributor Author

relate #1854

@tomsun28 tomsun28 requested a review from zqr10159 April 28, 2024 07:16
@github-actions github-actions bot added the home label Apr 28, 2024
@zqr10159 zqr10159 merged commit a10786c into master Apr 28, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend doc Improvements or additions to documentation home license new feature
Projects
Development

Successfully merging this pull request may close these issues.

None yet

2 participants