-
Notifications
You must be signed in to change notification settings - Fork 120
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
DP - field "hash" in resource information is unclear. #41
Comments
Yes it is a checksum. Options:
I would incline to either option 1 or 4. |
All are valid options if the standard itself is versioned - |
OK, i'm going to recommend option 1 and recommend md5 (following s3 etc). |
@relet thank-you very much for the issue and the suggestions here. As per discussion have resolved by clarifying this is md5 and there is only one hash field. |
* Added general/csv props * Added excel/db * Added json * Updated profile section * Updated the profile * Fixed spelling * Removed db for now; focus on csv/json/excel at the beggining * Updated CSV Dialect mention * Fixed quoteChar * Fixed doubleQuote * Fixed header * Updated introduction * Fixed the link * Update content/docs/specifications/table-dialect.md Co-authored-by: Peter Desmet <peter.desmet@inbo.be> * Update content/docs/specifications/table-dialect.md Co-authored-by: Peter Desmet <peter.desmet@inbo.be> * Update content/docs/specifications/table-dialect.md Co-authored-by: Peter Desmet <peter.desmet@inbo.be> * Added caseSensitiveHeader note --------- Co-authored-by: Peter Desmet <peter.desmet@inbo.be>
Is it a checksum?
Should it include the algorithm? Should it allow for several algorithms? Should it require certain algorithms?
or
?
The text was updated successfully, but these errors were encountered: