diff options
author | drh <> | 2024-08-20 23:11:28 +0000 |
---|---|---|
committer | drh <> | 2024-08-20 23:11:28 +0000 |
commit | 5648d717b15c1722ff19e7af1e55d68d1d172f1d (patch) | |
tree | 971f2c19f00ab2073124d4ecedaa7df3f0b42509 /src/attach.c | |
parent | 781163aa9dce91608ba33af479564f83dda1bd40 (diff) | |
parent | a0651b370706e58f9014453c5f36124ca8a738c9 (diff) | |
download | sqlite-5648d717b15c1722ff19e7af1e55d68d1d172f1d.tar.gz sqlite-5648d717b15c1722ff19e7af1e55d68d1d172f1d.zip |
Refactor the SrcItem object to move fields associated with subqueries out
into a separate object named Subquery. This reduces the size of the SrcItem
object by about 1/3rd and provides improved performance.
FossilOrigin-Name: 484bcd75bc95491d8540c791c1c4d40d996cb465839564662e14f98739699bf1
Diffstat (limited to 'src/attach.c')
-rw-r--r-- | src/attach.c | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/src/attach.c b/src/attach.c index 4a6a25bf0..76476685f 100644 --- a/src/attach.c +++ b/src/attach.c @@ -479,20 +479,21 @@ static int fixSelectCb(Walker *p, Select *pSelect){ if( NEVER(pList==0) ) return WRC_Continue; for(i=0, pItem=pList->a; i<pList->nSrc; i++, pItem++){ - if( pFix->bTemp==0 ){ - if( pItem->zDatabase ){ - if( iDb!=sqlite3FindDbName(db, pItem->zDatabase) ){ + if( pFix->bTemp==0 && pItem->fg.isSubquery==0 ){ + if( pItem->fg.fixedSchema==0 && pItem->u4.zDatabase!=0 ){ + if( iDb!=sqlite3FindDbName(db, pItem->u4.zDatabase) ){ sqlite3ErrorMsg(pFix->pParse, "%s %T cannot reference objects in database %s", - pFix->zType, pFix->pName, pItem->zDatabase); + pFix->zType, pFix->pName, pItem->u4.zDatabase); return WRC_Abort; } - sqlite3DbFree(db, pItem->zDatabase); - pItem->zDatabase = 0; + sqlite3DbFree(db, pItem->u4.zDatabase); pItem->fg.notCte = 1; + pItem->fg.hadSchema = 1; } - pItem->pSchema = pFix->pSchema; + pItem->u4.pSchema = pFix->pSchema; pItem->fg.fromDDL = 1; + pItem->fg.fixedSchema = 1; } #if !defined(SQLITE_OMIT_VIEW) || !defined(SQLITE_OMIT_TRIGGER) if( pList->a[i].fg.isUsing==0 |