google / google/googlesql

orderBy is not optional in Java client

Open
#84 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
2.6k
Forks
260
PR merge metrics
No merged PRs in 30d

Description

orderBy in ResolvedAnalyticFunctionGroup can be NULL and should thus be optional.

Expected resulted: We can set OrderBy as NULL
Current result: We get an error 'OrderBy needs to be set' when we create an ResolvedAnalyticFunctionGroup object

See https://github.com/google/zetasql/pull/82 for fix.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.