Conversation
Co-authored-by: Matthias Schur <107557548+MattSchur@users.noreply.github.com>
agoerler
left a comment
agoerler
left a comment
There was a problem hiding this comment.
We need to decide wether we use CDS type names Int32 or hana.Real or the constants of CdsBaseType. As we don't mention the CAP Java's CdsBaseType yet ion the documentation we could stick to the CDS type names.
Or we extend the table https://pages.github.tools.sap/cap/docs/java/cds-data#predefined-types and the table for the HANA types with a column for the CdsBaseType names.
Co-authored-by: Adrian Görler <adrian.goerler@sap.com>
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Here's a supportive code review enhanced by AI assistance. These insights offer suggestions and observations that may help guide your development process. Please consider them as recommendations rather than definitive requirements. Your expertise and judgment remain the most important factors in making decisions about your code. The AI analysis is simply here to provide additional perspective and support your work.
Model: claude-sonnet-4-20250514 | Prompt Tokens: 1463 | Completion Tokens: 384
java/cds-data.md
Outdated
| - **ceiling(x)**, **floor(x)**, **round(x)**: Return the same type as the input `x`. | ||
| - **min(x)**, **max(x)**, **sum(x)**: Aggregate functions return the same type as the argument `x`. | ||
| - **average(x)**: Returns `cds.Decimal` for exact numeric types and `cds.Double` for approximate numeric types. | ||
| - **count(x)**, **countdistinct(x)**: return `cds.Int64`. |
There was a problem hiding this comment.
Capitalize "Return" to maintain consistency with the capitalization pattern used in other function descriptions:
- **count(x)**, **countdistinct(x)**: Return `cds.Int64`.
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
4 participants
No description provided.