Taewoo Kim has posted comments on this change.

Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function
......................................................................


Patch Set 6:

For your comments, edit-distance-check_strings test case already contains that 
corner case. The latter two queries will do the early termination. I just 
checked it using "println".  

let $a := "Nalini Venkatasubramanian"
let $b := "Nalini Wekatasupramanian"
let $results :=
[
  edit-distance-check($a, $b, 3),
  edit-distance-check($b, $a, 3),
  edit-distance-check($a, $b, 2),
  edit-distance-check($b, $a, 2)
]
for $i in $results
return $i

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1481
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ibc8729c4514bb87c347dd7d50358fd897b769977
Gerrit-PatchSet: 6
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Taewoo Kim <[email protected]>
Gerrit-Reviewer: Chen Li <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Jianfeng Jia <[email protected]>
Gerrit-Reviewer: Steven Jacobs <[email protected]>
Gerrit-Reviewer: Taewoo Kim <[email protected]>
Gerrit-HasComments: No

Reply via email to