Fixed bug the initialization block (Insertion Level and Simple) #176
Conversation
update fork
atualizando fork
…ded test for insetion level when have more keys inside block.
| //only when there similar (left and right) nodes we proceed | ||
| if (rightSimilarBlock != null && leftSimilarBlock != null) { | ||
| //only when there similar (left or right) nodes we proceed | ||
| if (rightSimilarBlock != null || leftSimilarBlock != null) { |
There was a problem hiding this comment.
@leosoftlima pode proceder com apenas um sendo null? por que?
There was a problem hiding this comment.
@pauloborba @guilhermejccavalcanti Porque utilizando o operador || o algoritmo simples conseguirá abordar os blocos deletados, ou seja, Caso tenho 2 blocos na Base, no Left contenha dois blocos e com duas alterações nos mesmos e no right um bloco foi removido e o segundo bloco alterado. O resultado será um conflito para alteração e remoção. Diferentemente, se utilizar operador &&, O resultado será manter o bloco alterado.
There was a problem hiding this comment.
@pauloborba do jeito que estava antes, nunca havia conflito de deleção (um editou um bloco e o outro apagou). Mas note que uma edição pode ter sido tão significativa (menos de 70% de similaridade), que o algoritmo vai interpretar como deleção (com conflito) e adição (sem conflito) simultaneamente, algo parecido com o que acontece nos casos de renaming. Isto é, vai aparecer um bloco com conflito, e o outro sem conflito.
Fixed bug the initialization block (Insertion Level and Simple)
Correction