So should i change it and push the branch once again.
Akash
On Thu, Mar 17, 2011 at 11:03 PM, Andrew Hutchings <email address hidden> wrote: > Review: Approve > Good work, only 1 minor thing: > > calc_used_field_length( join_tab); > > shouldn't have the space before join_tab. > -- > https://code.launchpad.net/~sinha/drizzle/patch-621856/+merge/53537 > You are the owner of lp:~sinha/drizzle/patch-621856. >
« Back to merge proposal
So should i change it and push the branch once again.
Akash
On Thu, Mar 17, 2011 at 11:03 PM, Andrew Hutchings field_length( join_tab); /code.launchpad .net/~sinha/ drizzle/ patch-621856/ +merge/ 53537
<email address hidden> wrote:
> Review: Approve
> Good work, only 1 minor thing:
>
> calc_used_
>
> shouldn't have the space before join_tab.
> --
> https:/
> You are the owner of lp:~sinha/drizzle/patch-621856.
>