Copilot commented on code in PR #5606:
URL: https://github.com/apache/ignite-3/pull/5606#discussion_r2039043279
##########
modules/platforms/dotnet/Apache.Ignite/Sql/ColumnType.cs:
##########
@@ -29,58 +31,128 @@ namespace Apache.Ignite.Sql
[SuppressMessage("Design", "CA1027:Mark enums with FlagsAttribute",
Justification = "Not a flags enum.")]
public enum ColumnType
{
- /** Null. */
+ /// <summary>
+ /// Null.
+ /// </summary>
Null = 0,
- /** Boolean. */
+ /// <summary>
+ /// Boolean.
+ /// <para />
+ /// SQL type: <c>BOOLEAN</c>, .NET type: <see cref="bool"/>.
+ /// </summary>
Boolean = 1,
- /** 8-bit signed integer. */
+ /// <summary>
+ /// 8-bit signed integer.
+ /// <para />
+ /// SQL type: <c>TINYINT</c>, .NET type: <see cref="sbyte"/>.
+ /// </summary>
Int8 = 2,
- /** 16-bit signed integer. */
+ /// <summary>
+ /// 16-bit signed integer.
+ /// <para />
+ /// SQL type: <c>SMALLINT</c>, .NET type: <see cref="short"/>.
+ /// </summary>
Int16 = 3,
- /** 32-bit signed integer. */
+ /// <summary>
+ /// 32-bit signed integer.
+ /// <para />
+ /// SQL type: <c>INTEGER</c>, .NET type: <see cref="int"/>.
+ /// </summary>
Int32 = 4,
- /** 64-bit signed integer. */
+ /// <summary>
+ /// 64-bit signed integer.
+ /// <para />
+ /// SQL type: <c>BIGINT</c>, .NET type: <see cref="long"/>.
+ /// </summary>
Int64 = 5,
- /** 32-bit single-precision floating-point number. */
+ /// <summary>
+ /// 32-bit single-precision floating-point number.
+ /// <para />
+ /// SQL type: <c>REAL</c>, .NET type: <see cref="float"/>.
+ /// </summary>
Float = 6,
- /** 64-bit double-precision floating-point number. */
+ /// <summary>
+ /// 64-bit single-precision floating-point number.
Review Comment:
The description for the 'Double' enum incorrectly states 'single-precision';
it should be '64-bit double-precision floating-point number.'
```suggestion
/// 64-bit double-precision floating-point number.
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
