yuqi1129 commented on PR #12364:
URL: https://github.com/apache/gravitino/pull/12364#issuecomment-5237450530
@shaoyu-li
After a more thorough analysis, I would like to revise my earlier
assessment. I believe this change should be classified as an improvement rather
than a bug fix. Although the current index names may be debatable, they are
valid in MySQL and do not cause functional issues.
Traditionally, unless there is a critical bug, we avoid changing database
scripts in patch releases. Otherwise, we need to introduce and maintain
additional upgrade paths between patch and major versions. As of 1.3.0, we have
not made schema changes between patch releases.
Therefore, I have some concerns about introducing these changes in 1.3.1.
Would it be possible to limit this PR to 2.0.0? For your existing 1.3.0
deployment, I suggest treating this as an internal improvement and maintaining
the corresponding migration internally.
Looking forward to hearing more about this. Thanks.
--
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]
