Re: [Zorba-coders] [Merge] lp:~markos-za/zorba/indexes into lp:zorba

2011-11-01 Thread Matthias Brantner
Review: Needs Fixing

some parameters of the probe-index-range-general#7 function are not documented.
Also, should this function be variadic?

-- 
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/79334
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~markos-za/zorba/indexes into lp:zorba

2011-11-01 Thread Markos Zaharioudakis
 some parameters of the probe-index-range-general#7 function are not
 documented.
 Also, should this function be variadic?

I have added the missing params.

No, the function is not variadic because it applies to general indexes only, 
whicg have only one key column.
-- 
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/79334
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~markos-za/zorba/indexes into lp:zorba

2011-11-01 Thread Matthias Brantner
Review: Needs Fixing

Markos, the branch is now conflicting with the trunk. I have tried to resolve 
it but I don't have permissions to push to this branch.
-- 
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/79334
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~markos-za/zorba/indexes into lp:zorba

2011-11-01 Thread Markos Zaharioudakis
Review: Approve


-- 
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/79334
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~markos-za/zorba/indexes into lp:zorba

2011-11-01 Thread Matthias Brantner
Review: Approve


-- 
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/79334
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp


Re: [Zorba-coders] [Merge] lp:~markos-za/zorba/indexes into lp:zorba

2011-10-04 Thread Markos Zaharioudakis
Review: Approve


-- 
https://code.launchpad.net/~markos-za/zorba/indexes/+merge/78060
Your team Zorba Coders is subscribed to branch lp:zorba.

-- 
Mailing list: https://launchpad.net/~zorba-coders
Post to : zorba-coders@lists.launchpad.net
Unsubscribe : https://launchpad.net/~zorba-coders
More help   : https://help.launchpad.net/ListHelp