LabKey / platform

A set of modules that provide core functionality for LabKey Server
Apache License 2.0
7 stars 7 forks source link

Revert PK add to core.Containers #5980

Closed labkey-adam closed 1 week ago

labkey-adam commented 2 weeks ago

Rationale

My attempt to designate core.Containers.RowId as this table's PK failed. Too many code paths assume EntityId is the (pseudo) PK... for example, lookups to the Containers table. Best to revert it for now. Better would be to designate EntityId as an actual PK, but it's a nullable column so this is not a trivial change.

Related Pull Requests