Skip to content

Commit 0d76e66

Browse files
committed
chore: remove old code
1 parent 81760f6 commit 0d76e66

File tree

1 file changed

+0
-15
lines changed

1 file changed

+0
-15
lines changed

src/cloudsync.c

Lines changed: 0 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -1711,13 +1711,6 @@ int cloudsync_refill_metatable (sqlite3 *db, cloudsync_context *data, const char
17111711
char *pkdecode = dbutils_text_select(db, sql);
17121712
char *pkdecodeval = (pkdecode) ? pkdecode : "cloudsync_pk_decode(pk, 1) AS rowid";
17131713
cloudsync_memory_free(sql);
1714-
1715-
/* not needed in VERSION 2
1716-
sql = cloudsync_memory_mprintf("SELECT group_concat('\"' || format('%%w', name) || '\"' || ' = cloudsync_pk_decode(pk, ' || pk || ')', ' AND ') FROM pragma_table_info('%q') WHERE pk>0 ORDER BY pk;", table_name);
1717-
char *pkonclause = dbutils_text_select(db, sql);
1718-
char *pkonclauseval = (pkonclause) ? pkonclause : "rowid = cloudsync_pk_decode(pk, 1) AS rowid";
1719-
cloudsync_memory_free(sql);
1720-
*/
17211714

17221715
sql = cloudsync_memory_mprintf("SELECT cloudsync_insert('%q', %s) FROM (SELECT %s FROM \"%w\" EXCEPT SELECT %s FROM \"%w_cloudsync\");", table_name, pkvalues_identifiers, pkvalues_identifiers, table_name, pkdecodeval, table_name);
17231716
int rc = sqlite3_exec(db, sql, NULL, NULL, NULL);
@@ -1726,12 +1719,6 @@ int cloudsync_refill_metatable (sqlite3 *db, cloudsync_context *data, const char
17261719

17271720
// fill missing colums
17281721
// for each non-pk column:
1729-
1730-
// VERSION 2 (October 15, 2025)
1731-
// previous query WAS
1732-
// sql = cloudsync_memory_mprintf("SELECT cloudsync_pk_encode(%s) FROM \"%w\" LEFT JOIN \"%w_cloudsync\" ON %s AND \"%w_cloudsync\".col_name = ? WHERE \"%w_cloudsync\".db_version IS NULL", pkvalues_identifiers, table_name, table_name, pkonclauseval, table_name, table_name);
1733-
// but it takes hours with large tables (100,000 rows, 25 columns, of which 18 was primary keys)
1734-
//
17351722
// The new query does 1 encode per source row and one indexed NOT-EXISTS probe.
17361723
// The old plan does many decodes per candidate and can’t use an index to rule out matches quickly—so it burns CPU and I/O.
17371724

@@ -1768,8 +1755,6 @@ int cloudsync_refill_metatable (sqlite3 *db, cloudsync_context *data, const char
17681755
if (rc != SQLITE_OK) DEBUG_ALWAYS("cloudsync_refill_metatable error: %s", sqlite3_errmsg(db));
17691756
if (pkclause_identifiers) cloudsync_memory_free(pkclause_identifiers);
17701757
if (pkdecode) cloudsync_memory_free(pkdecode);
1771-
// not needed in VERSION 2
1772-
// if (pkonclause) cloudsync_memory_free(pkonclause);
17731758
if (vm) sqlite3_finalize(vm);
17741759
return rc;
17751760
}

0 commit comments

Comments
 (0)