Hi Mikhail,<br>
<br>
Yes, the specs are fuzzy on this as we used to have an implementation in the forerunner to XRootD but it never got ported across. So, the specs remained fuzzy simply because we had no reason to flesh them out. I suppose, for now, never specify kXR_compress and your implementation should be safe. In the mean time, I will put compression elaboration on the list. Notethat cptype is a four byte field and if it’s null terminated (not clear it is – I have to look deeper) that leave only three characters for compression type. In any case, the idea was it was going to be like “lz4”.<br>
Andy <br>
From: Mikhail Ivchenko <br>
Sent: Tuesday, June 05, 2018 9:06 AM<br>
To: xrootd/xrootd <br>
Cc: Subscribed <br>
Subject: [xrootd/xrootd] specs: cpsize and cptype fields lack some documentation (#724)<br>
<br>
Specs do not clarify what&#39;s the unit of the size and how type is defined (is it the name of the compression algorithm like deflate or lz4?).<br>
<br>
As far as I got, compressed files are not commonly used in XRootD, however, I&#39;m working on the XRootD protocol implementation in Go language and would like to clarify these details in doc string if they are available.<br>
<br>
—<br>
You are receiving this because you are subscribed to this thread.<br>
Reply to this email directly, view it on GitHub, or mute the thread.<br>


<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">&mdash;<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/xrootd/xrootd/issues/724#issuecomment-394866751">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AD7YjrJIrrWQaU5lRcEMMc6gdUX7g3DXks5t5vhKgaJpZM4UbHvo">mute the thread</a>.<img src="https://github.com/notifications/beacon/AD7YjkBz5ElbxNwyjl4dWf5PdwmvJ0Hiks5t5vhKgaJpZM4UbHvo.gif" height="1" width="1" alt="" /></p>
<!cript type="application/ld+json">{"@context":"http://schema.org","@type":"EmailMessage","potentialAction":{"@type":"ViewAction","target":"https://github.com/xrootd/xrootd/issues/724#issuecomment-394866751","url":"https://github.com/xrootd/xrootd/issues/724#issuecomment-394866751","name":"View Issue"},"description":"View this Issue on GitHub","publisher":{"@type":"Organization","name":"GitHub","url":"https://github.com"}}</script>
<!cript type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/xrootd/xrootd","title":"xrootd/xrootd","subtitle":"GitHub repository","main_image_url":"https://assets-cdn.github.com/images/email/message_cards/header.png","avatar_image_url":"https://assets-cdn.github.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/xrootd/xrootd"}},"updates":{"snippets":[{"icon":"PERSON","message":"@abh3 in #724: Hi Mikhail,\n\nYes, the specs are fuzzy on this as we used to have an implementation in the forerunner to XRootD but it never got ported across. So, the specs remained fuzzy simply because we had no reason to flesh them out. I suppose, for now, never specify kXR_compress and your implementation should be safe. In the mean time, I will put compression elaboration on the list. Notethat cptype is a four byte field and if it’s null terminated (not clear it is – I have to look deeper) that leave only three characters for compression type. In any case, the idea was it was going to be like “lz4”.\nAndy \nFrom: Mikhail Ivchenko \nSent: Tuesday, June 05, 2018 9:06 AM\nTo: xrootd/xrootd \nCc: Subscribed \nSubject: [xrootd/xrootd] specs: cpsize and cptype fields lack some documentation (#724)\n\nSpecs do not clarify what's the unit of the size and how type is defined (is it the name of the compression algorithm like deflate or lz4?).\n\nAs far as I got, compressed files are not commonly used in XRootD, however, I'm working on the XRootD protocol implementation in Go language and would like to clarify these details in doc string if they are available.\n\n—\nYou are receiving this because you are subscribed to this thread.\nReply to this email directly, view it on GitHub, or mute the thread.\n"}],"action":{"name":"View Issue","url":"https://github.com/xrootd/xrootd/issues/724#issuecomment-394866751"}}}</script>
<!cript type="application/ld+json">{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "37567f93-e2a7-4e2a-ad37-a9160fc62647",
"title": "Re: [xrootd/xrootd] specs: cpsize and cptype fields lack some documentation (#724)",
"sections": [
{
"text": "",
"activityTitle": "**Andrew Hanushevsky**",
"activityImage": "https://assets-cdn.github.com/images/email/message_cards/avatar.png",
"activitySubtitle": "@abh3",
"facts": [

]
}
],
"potentialAction": [
{
"name": "Add a comment",
"@type": "ActionCard",
"inputs": [
{
"isMultiLine": true,
"@type": "TextInput",
"id": "IssueComment",
"isRequired": false
}
],
"actions": [
{
"name": "Comment",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueComment\",\n\"repositoryFullName\": \"xrootd/xrootd\",\n\"issueId\": 724,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close issue",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"IssueClose\",\n\"repositoryFullName\": \"xrootd/xrootd\",\n\"issueId\": 724\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/xrootd/xrootd/issues/724#issuecomment-394866751"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 342653928\n}"
}
],
"themeColor": "26292E"
}</script><br>
<hr>
<p align="left">
Use REPLY-ALL to reply to list</p>
<p align="center">To unsubscribe from the XROOTD-DEV list, click the following link:<br>
<a href="https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1" target="_blank">https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1</a>
</p>