-
Notifications
You must be signed in to change notification settings - Fork 372
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
Update schema versioning tools #117
base: master
Are you sure you want to change the base?
Update schema versioning tools #117
Conversation
federico-razzoli
commented
May 29, 2024
- Update Flyway URL
- Add ByteBase
https://github.com/bytebase/bytebase/blob/main/LICENSE says this is under the "MIT Expat" license (except for enterprise), so that looks ok to me. |
Thank you @dveeden -- I honeslty don't dig the
which in turn requires as to agree to https://bytebase.com/terms. I am not a lawyer and cannot verify that these terms comply with any identifyable OSS license. |
The "enterprise" license applies to:
I don't know what to make of it. Are these directories important? Not important? Will I as a user be using those directories? |
README.md
Outdated
@@ -115,7 +115,8 @@ This list accepts and encourages pull requests. See [CONTRIBUTING](https://githu | |||
|
|||
*Tools to support MySQL-related development* | |||
|
|||
- [Flywaydb](http://flywaydb.org/getstarted/) - Database migrations; Evolve your database schema easily and reliably across all your instances | |||
- [ByteBase](https://www.bytebase.com/) - Validate and version schema changes (amongst other things) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure yet about the license, but I prefer replacing the URL with the GitHub project page, seeing that the product URL is more on the marketing side.
Side note: we should probably do the same for many other projects here.
- [ByteBase](https://www.bytebase.com/) - Validate and version schema changes (amongst other things) | |
- [ByteBase](https://github.com/bytebase/bytebase) - Validate and version schema changes (amongst other things) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@federico-razzoli could you apply this suggestion or do you have any objections?
README.md
Outdated
@@ -115,7 +115,8 @@ This list accepts and encourages pull requests. See [CONTRIBUTING](https://githu | |||
|
|||
*Tools to support MySQL-related development* | |||
|
|||
- [Flywaydb](http://flywaydb.org/getstarted/) - Database migrations; Evolve your database schema easily and reliably across all your instances | |||
- [ByteBase](https://www.bytebase.com/) - Validate and version schema changes (amongst other things) | |||
- [Flywaydb](https://flywaydb.org/) - Database migrations; Evolve your database schema easily and reliably across all your instances |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Best to use a GitHub link for Flyway. This is done in #129 as well.
ByteBase has an Enterprise and an open edition. Yes, it's fine to link the GitHub repo (both if you do it for this case only or for all projects). |
Thanks @federico-razzoli ! But it's still unclear to me. It's not dual licensed. The "edition" is the use of specific folders, but I do not understand what use cases do and what use cases do not fall under using those specific folders. It's really confusing and not clarified in the docs/manual. Which puts the burden on us as reviewers to understand what they mean... |
Let's tag @tianzhou for a clarification. |
@shlomi-noach @federico-razzoli it's fair to link the github repo. The enterprise "edition" is for gating the license related logic. The Enterprise license explicitly disallows to modify the licensing code. (e.g. func isEnterprise() { return true }; |
@tianzhou thank you; I confess I do not fully understand. Is the nature of the enterprise license to control the change of the license? How about all the files under specified directories, such as |
Right, it's for gating the enterprise related features.
It's an oversight. We just removed this bytebase/bytebase#12198 |
Thank you @tianzhou. Any other reviewers who have an opinion? I'm still not a lawyer and still cannot firmly conclude that the terms of this proposed code comply with OSS licensing. |
@shlomi-noach None of the people who commented until now is a lawyer, so I think that the decision can only be a "political" one. I proposed the project because - despite the uncommon practice you highlighted - it seems to me open core, just like MySQL. In case it isn't for some reason, that would be a bug, I think. But for such a corner case, only you can decide to approve or reject, imho. |
My vote would be to include this. I think the intention of the license is ok. This is not a strong vote. However I would also like to suggest Bytebase to consider splitting the enterprise code into a separate repo or to implement some other solution that would make this more clear an avoid any future issues. |
README.md
Outdated
@@ -115,7 +115,8 @@ This list accepts and encourages pull requests. See [CONTRIBUTING](https://githu | |||
|
|||
*Tools to support MySQL-related development* | |||
|
|||
- [Flywaydb](http://flywaydb.org/getstarted/) - Database migrations; Evolve your database schema easily and reliably across all your instances | |||
- [ByteBase](https://www.bytebase.com/) - Validate and version schema changes (amongst other things) | |||
- [Flywaydb](https://flywaydb.org/) - Database migrations; Evolve your database schema easily and reliably across all your instances |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Best to use a GitHub link for Flyway. This is done in #129 as well.
README.md
Outdated
@@ -115,7 +115,8 @@ This list accepts and encourages pull requests. See [CONTRIBUTING](https://githu | |||
|
|||
*Tools to support MySQL-related development* | |||
|
|||
- [Flywaydb](http://flywaydb.org/getstarted/) - Database migrations; Evolve your database schema easily and reliably across all your instances | |||
- [ByteBase](https://www.bytebase.com/) - Validate and version schema changes (amongst other things) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@federico-razzoli could you apply this suggestion or do you have any objections?