yanivru commented on a change in pull request #1597:
URL: https://github.com/apache/avro/pull/1597#discussion_r827449126
##########
File path: lang/csharp/src/apache/main/Schema/Field.cs
##########
@@ -105,6 +105,29 @@ public enum SortOrder
/// </summary>
internal static JTokenEqualityComparer JtokenEqual = new
JTokenEqualityComparer();
+ /// <summary>
+ /// Initializes a new instance of the <see cref="Field"/> class.
+ /// </summary>
+ /// <param name="schema">schema for the field type.</param>
+ /// <param name="name">name of the field.</param>
+ /// <param name="aliases">list of aliases for the name of the
field.</param>
+ /// <param name="pos">position of the field.</param>
+ /// <param name="doc">documentation for the field.</param>
+ /// <param name="defaultValue">field's default value if it
exists.</param>
+ /// <param name="sortorder">sort order of the field.</param>
+ /// <param name="customProperties">dictionary that provides access to
custom properties.</param>
+ public Field(Schema schema,
Review comment:
I wanted to add default values to non mandatory parameters. I had to
change their order (switch between aliases and pos), and didn't want to make a
breaking change.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
