D10 - Status Report primary key not set for views_url_alias [#3396154]

Problem/Motivation

after d10 upgrade, status report error.

Transaction isolation level
READ-COMMITTED
For this to work correctly, all tables must have a primary key. The following table(s) do not have a primary key: views_url_alias. See the setting MySQL transaction isolation level page for more information.
https://www.drupal.org/docs/getting-started/system-requirements/setting-...

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comment	File	Size	Author
#6	3396154.patch	15.08 KB	neptune-dc

Issue fork views_url_alias-3396154

Show commands

Start within a Git clone of the project using the version control instructions.

Add & fetch this issue fork’s repository

Or, if you do not have SSH keys set up on git.drupalcode.org:

Add & fetch this issue fork’s repository

3396154-d10---status changes, plain diff MR !15
Check out this branch for the first time

Check out existing branch, if you already have it locally

About issue forks

Comments

Comment #1

23 October 2023 at 16:26

NicholasS created an issue. See original summary.

Comment #2

23 October 2023 at 16:27

NicholasS opened merge request !15

Comment #3

nicholass

English

commented 23 October 2023 at 16:36

Humm. so this module entity_id can have duplicates (Node/Taxonomy IDs) and alias also does not work for the primary key.... I think we have to add a new field to act as a primary key

Comment #4

nicholass

English

commented 23 October 2023 at 18:38

Status:

Active

» Needs review

Comment #5

nicholass

English

commented 25 October 2023 at 16:44

Status:	Needs review	» Closed (outdated)
Related issues:		+#3396617: Timeout on large sites nodes during install

Closing since this fix is going into just a single issue

Comment #6

neptune-dc commented 28 December 2023 at 04:51

Status	File	Size
new	3396154.patch	15.08 KB

I needed this patch for my site, so I compiled NicholasS's changes here. In an ideal world, his changes wouldn't take more than two months to merge.

This patch worked for me with the latest version of the module on Drupal 10.

Thanks a lot.

Comment #7

dstorozhuk

Chicago 🇺🇸, Toronto 🇨🇦, Kyiv 🇺🇦

commented 22 March 2024 at 12:50

I think this issue still valid. Patch from neptune-dc Looks pretty big. Can you, neptune-dc, please create a pull request and remove all non task related issue from you proposal? Or better to cooperate with NicholasS and add changes to his MR.