Skip to content

Fix missing comma in entry_vendor_task index definition#13966

Open
mehmetozzen wants to merge 1 commit into
kaltura:Venus-22.20.0from
mehmetozzen:fix/entry-vendor-task-missing-comma
Open

Fix missing comma in entry_vendor_task index definition#13966
mehmetozzen wants to merge 1 commit into
kaltura:Venus-22.20.0from
mehmetozzen:fix/entry-vendor-task-missing-comma

Conversation

@mehmetozzen

Copy link
Copy Markdown

CREATE TABLE fails with a syntax error on MySQL 5.7 fresh installs due to
a missing comma after the reach_profile_queue_time index definition.

Pull Request Checklist

Please complete the following before submitting:

General notes -

  • I have tested the changes locally.
  • I have written unit tests where applicable.
  • I have updated documentation where needed.
  • I have added comments to complex code.
  • This PR follows the coding style guidelines.
  • I have updated release notes with new feature

New Kaltura Types

  • I have created new clients
  • I have notified related apps - KMCNG / KMS / EP .... about new clients

New Kaltura Services / Actions

  • I have added a deployment script

Questions

  1. What is the purpose of this PR?

    • Fixes a MySQL syntax error in the entry_vendor_task table DDL where a missing comma between index definitions caused fresh installation scripts to fail.
  2. Does this change affect production code or infrastructure?

    • Yes
    • No
  3. If yes, what is the rollback plan?

    • N/A (It's a one-character syntax fix for a table definition).

CREATE TABLE fails with a syntax error on MySQL 5.7 fresh installs due
to
a missing comma after the reach_profile_queue_time index definition.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant