search

slipsoft / slipdb

Distributed index based search engine.
4 stars 0 forks source link

Désormais possible de flag les lignes supprimées #80

Closed SylvainJoube closed 5 years ago

SylvainJoube commented 5 years ago

J'ai fait comme Nicolas a dit, j'ai fait une feature pour ne pas avoir à merge sylvain-master dans master ! :3

Cette feature est l'ajout de flag pour savoir quand une ligne est supprimée

SylvainJoube commented 5 years ago

J'avoue, je suis grave coquin, je viens de commit un truc dont j'avais besoin en speed ^^'

n-peugnet commented 5 years ago

J'avoue, je suis grave coquin, je viens de commit un truc dont j'avais besoin en speed ^^'

non pas grave tfacon je t'avais pas validé héhé

SylvainJoube commented 5 years ago

ok c'est marrant, là sans y avoir réfléchi beaucoup je n'aurais pas fait ça de la même manière mais bon j'y ai pas tellement réfléchi justement.

Par contre ça aurait été cool que tu fasses un vrai test avec au moins une assertion de la taille que tu attends pour être sur qu'on ne casse rien par la suite

J'ai fait boolean checkLinesNumber = table.testCheckLinesNumber(); dans le code, (dans NewLoaderAndIndexTest), c'est ps un arrertEquals mais ça throw une exception en cas de non égalité ^^ J'ai bien bien vérifié que ça n'allait pas tout faire foirer ^^

n-peugnet commented 5 years ago

J'ai fait boolean checkLinesNumber = table.testCheckLinesNumber(); dans le code, (dans NewLoaderAndIndexTest), c'est ps un arrertEquals mais ça throw une exception en cas de non égalité ^^

hmm je voulais dire un test du genre

assertEquals(1000000, realNumberOfLines);
table.flagline(3);
assertEquals(999999, realNumberOfLines);

c'est du pseudo code bien-sûr

SylvainJoube commented 5 years ago

J'avoue, j'avoue, mais je manque de temps, et j'ai besoin de beaucoup modifier les jeux de tests que j'utilsie ^^' Mais grave, il faudrait faire ça si je voulais vraiment bien coder et que j'avais le temps ^^"

n-peugnet commented 5 years ago

J'avoue, j'avoue, mais je manque de temps, et j'ai besoin de beaucoup modifier les jeux de tests que j'utilsie ^^' Mais grave, il faudrait faire ça si je voulais vraiment bien coder et que j'avais le temps ^^"

En fait le vrai problème c'est qu'on aurait du mettre en place un truc pour pouvoir switcher de jeux de tests en local

SylvainJoube commented 5 years ago

Ahhhhhhhhh en fait, y a un gros bug ^^'

SylvainJoube commented 5 years ago

Enfin, un petit gros bug bête, de toute façon, je vais faire une PR et merge sylvain-master dans master ^^