4paradigm / 4paradigm/OpenMLDB

Remove the memcpy for UnsafeRowOpt

Aperta
#1,893 0 commenti 0 reazioni 1 assegnatario Rivendicata da @tobegit3hub Vedi su GitHub
enhancement
Lingua principale
C++
Stelle
1.7k
Fork
331
Merge medio
12g 12h
PR unite (30g)
1

Descrizione

Now we need to memcpy the entire row for UnsafeRowOpt because of usage of row header. It would be great to remove the memcpy for UnsafeRowOpt by not depending on row header in some cases.

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Look at the UnsafeRowOpt implementation, likely in a file like core/row_opt.cpp or similar. Understand the current row header usage and memcpy. Identify cases where the header dependency can be avoided. Test changes with existing unit tests for row operations and performance benchmarks.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
cpp
Ambito
databases, performance
Tipo di issue
Refactoring
Difficoltà
4/5
Tempo stimato
3-5 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
35/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.