4paradigm / 4paradigm/OpenMLDB

Remove the memcpy for UnsafeRowOpt

Ouverte
#1,893 0 commentaires 0 réactions 1 personne assignée Réclamée par @tobegit3hub Voir sur GitHub
enhancement
Langage dominant
C++
Étoiles
1.7k
Forks
331
Merge moyen
12 j 12 h
PR mergées (30 j)
1

Description

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.

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

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.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
cpp
Domaine
databases, performance
Type d'issue
Refactorisation
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
35/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.