search

ekinoguz / sprout-database

Database project of a graduate class.
0 stars 0 forks source link

Examine their VarChar Test Case #23

Closed diedthreetimes closed 11 years ago

diedthreetimes commented 11 years ago

it is the extra test case 2.

What is it supposed to do? More importantly, does it pass?

I notice that a scan of a single key returns

488 488 462 462 436 436 410 410 384 384 358 358 332 332 306 306 280 280 254 254 228 228 202 202 176 176 150 150 124 124 98 98 72 72 46 46 20 20

Is this correct?

ekinoguz commented 11 years ago

I added extra check for this case, it looks correct except the order of rids. We are adding 20, 46, 72, ... order and returning backwards. If this is not a problem, then we are fine

diedthreetimes commented 11 years ago

I still don't understand why more than one item is returned. Any headway on that On May 24, 2013 12:39 PM, "ekin oguz" notifications@github.com wrote:

I added extra check for this case, it looks correct except the order of rids. We are adding 20, 46, 72, ... order and returning backwards. If this is not a problem, then we are fine

— Reply to this email directly or view it on GitHubhttps://github.com/ekinoguz/cs222-database/issues/23#issuecomment-18422081 .

ekinoguz commented 11 years ago

Because we are adding keys as "count = ((i-1) % 26) + 1" which means there are duplicate keys in mod 26

diedthreetimes commented 11 years ago

Awesome! On May 24, 2013 1:17 PM, "ekin oguz" notifications@github.com wrote:

Because we are adding keys as "count = ((i-1) % 26) + 1" which means there are duplicate keys in mod 26

— Reply to this email directly or view it on GitHubhttps://github.com/ekinoguz/cs222-database/issues/23#issuecomment-18423754 .