[tarantool-patches] Re: [PATCH v1 1/1] sql: fix assertion on invalid PK column name
v.shpilevoy at tarantool.org
Thu Jul 12 13:28:36 MSK 2018
On 12/07/2018 13:03, Kirill Shcherbatov wrote:
>> I know, that removal of assertion always 'fixes' it, but please,
>> either fix it without removal (and for me it is not obvious why it
>> should be removed), or explain, why it is invalid.
> I've included this assertion during refactoring and there is no root causes to
> keep it, but as we verbally discuss, let's make it working correctly appending
> additional pCol == NULL OR condition that code below correctly deal with.
> If specified PK column modifier refers nonexistent column,
> lookup by name keeps NULL column pointer unchanged that should
> be accounted in assert.
> Close #3522.
More information about the Tarantool-patches