Re: aggregate test case typo

Seconded.

(I know you said to speak up if you disagree, but I approve of the change)

Paul

On Wed, Oct 24, 2012 at 12:08 PM, Lee Feigenbaum <lee@thefigtrees.net> wrote:
> Please speak up if you disagree with:
>
> PROPOSED: Update the .srx file for :agg-empty-group to include the ?x
> variable in the head section & re-approve the test
>
> Lee
>
>
> On 10/24/2012 11:59 AM, Gregory Williams wrote:
>>
>> Gunnar Grimnes mentioned to me on irc that he noticed something strange in
>> the :agg-empty-group aggregate test. The query projects ?x (the aggregate
>> grouping var) and ?max (the result of the MAX aggregate), but the results
>> file only has ?max in the srx <head> section.
>>
>> Andy and I both agree that this is wrong, and the results file should also
>> include <variable name="x">. However, the test has already been approved, so
>> probably needs WG consensus on fixing it.
>>
>> thanks,
>> .greg
>>
>>
>>
>
>

Received on Wednesday, 24 October 2012 16:30:38 UTC