Result for aggregation function, e.g. row count

[ingted]

Will we support to have value returned within a result set?

[UPDATE] I found that metrics is not the aggregation value...

Related test case:
count test

[ingted]

Commit d293eba resolved this.

[ingted]

Oops, count(*) is not supported so far...

[ingted]

Another issue:

        let preparedQueries = StaticQueryParser.ParseBatch(_core, "SELECT COUNT(1) FROM testSch", userParameters)
        let preparedQuery = preparedQueries.Item 0
        
        let queryResultCollection = _core.Query.ExecuteQuery(preLogin, preparedQuery)
        equals 1 queryResultCollection.Collection.Count
        equals 1 queryResultCollection.RowCount

queryResultCollection.Collection.Count is correct since the row with count value returned. Collection Count is 1, passed.
Not sure if queryResultCollection.RowCount should be 1 or?

[ingted]

Last possible issue(?):

if (value == null)
{
    if (property.PropertyType.IsValueType && Nullable.GetUnderlyingType(property.PropertyType) == null)
    {
        continue; // Skip setting value if property is non-nullable value type
    }
    else
    {
        property.SetValue(obj, null);
    }
}
else
{
    property.SetValue(obj, Convert.ChangeType(value, Nullable.GetUnderlyingType(property.PropertyType) ?? property.PropertyType));
}

Seems like "continue" caused MapTo a type like

class Count {
    int count
}

Doesn't work? (need more test...)

[ingted]

So far we have this in test case:

        let queryResultCollection = _core.Query.ExecuteQuery(preLogin, preparedQuery)
        equals 1 queryResultCollection.Collection.Count

        let queryDocList = ((queryResultCollection.Collection.Item 0) :?> KbQueryDocumentListResult).Rows
        equals 1 queryDocList.Count
        equals "2" queryDocList[0].Values[0] //inserted two documents

[NTDLS]

MaptTo<> only supports setting properties, not fields so this works:

When grouping, the "affected row count" should return the number of grouped rows, not the total underlying row count, so I thinkk what we have is good.

[ingted]

Do (will) we support count(*) or not (will not)?

[NTDLS]

We do now! 😃