Youssef Hatem

Results 9 comments of Youssef Hatem

This is also required to properly derive (named) grouping columns, for example: ```sql SELECT x, MAX(c) FROM t GROUP BY a AS x; ``` The method `GroupByExpression#computeRequestOrdering` should return `(a)`...

We should cover `Set.of` -> `ImmutableSet.of()` as pointed at here https://github.com/FoundationDB/fdb-record-layer/pull/2464#discussion_r1478676669

Thanks @ScottDugas for the reminder, the server does not support multi-statement transaction. i.e. this is only supported in embedded mode at the moment.

That's correct, the plan generator is not looking at the built-in function `ExistsFn` and instead it constructs the `ExistsValue` by itself: https://github.com/FoundationDB/fdb-record-layer/blob/b668b35fff4d37c7b1d0f7bf38007a2ca3fd9a4c/fdb-relational-core/src/main/java/com/apple/foundationdb/relational/recordlayer/query/visitors/ExpressionVisitor.java#L385-L398 Sp, I think you can do whatever you...

Closing this, rollups is already implemented in https://github.com/FoundationDB/fdb-record-layer/pull/3266

> I believe this is basically duck typing, which is an interesting approach. I don't think this is correct. The type that is built by the query processing is done...

[filtered-index-bug-repro.yaml](https://github.com/user-attachments/files/23826718/filtered-index-bug-repro.yaml) YAML test file that reproduces the issue.

The [README](https://github.com/FoundationDB/fdb-record-layer/blob/main/README.md) file is also out of date: > Queries - The Record Layer does not provide a query language, however it provides query APIs with the ability to scan,...