You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Kalle Niemitalo (Jira)" <ji...@apache.org> on 2022/07/03 10:03:00 UTC
[jira] [Created] (AVRO-3567) XML doc of methods of SchemaNormalization link to private method Fingerprint64(byte[])
Kalle Niemitalo created AVRO-3567:
-------------------------------------
Summary: XML doc of methods of SchemaNormalization link to private method Fingerprint64(byte[])
Key: AVRO-3567
URL: https://issues.apache.org/jira/browse/AVRO-3567
Project: Apache Avro
Issue Type: Improvement
Components: csharp, doc
Affects Versions: 1.11.0
Reporter: Kalle Niemitalo
In the Avro.SchemaNormalization class of the Avro C# library, the XML documentation comments of the following methods describe the behaviour by linking to the {{Fingerprint64(byte[])}} method:
* public static byte[] Fingerprint(string fpName, byte[] data)
{code:C#}
/// <para>If <i>fpName</i> equals the string
/// <code>"CRC-64-AVRO"</code>, then the result of <see cref="Fingerprint64(byte[])"/> is
/// returned in little-endian format.
/// </para>
{code}
* public static long ParsingFingerprint64(Schema s)
{code:C#}
/// Returns <see cref="Fingerprint64(byte[])"/> applied to the parsing canonical form of the supplied schema.
{code}
The documentation of the Fingerprint64(byte[]) method then says it computes "the 64-bit Rabin Fingerprint (as recommended in the Avro spec)". However, because the Fingerprint64(byte[]) method is private, Microsoft Visual Studio does not readily display this documentation to developers.
Please change the documentation of the Fingerprint(string, byte[]) and ParsingFingerprint64(Schema) methods to mention "the 64-bit Rabin Fingerprint" directly, instead of linking to the private method. Alternatively, make the method public.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)