Not at all. In line XrdXrootdXeq.cc:396 a specific check is made whether check sums are allowed to be computed using a local method. If so, a call is made to do that which, if not supported, will fail the request (i.e. one should not configure local computation if it cannot be done). Otherwise, we fall through to check if a script is available to do this (line 400) and if there is one it is invoked. So, if you specified that the checksum is to be computed using a script (or program) then the whole thing works regardless of whether or not extended attributes are supported. If you said you wanted local computation then the default implementation requires that extended attributes be supported, I suspect that the notion was that local computation should happen no matter what. The answer is that if you supplied a checksum manager plug-in then that might be possible (DPM does this). Otherwise, it's a no-no. I agree that the documentation should be clearer on this point. That said, it's hard to cover all the cases since there are way too many implicit variables that control whether or not a checksum can be computed at all.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
{"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 #769: Not at all. In line XrdXrootdXeq.cc:396 a specific check is made whether check sums are allowed to be computed using a local method. If so, a call is made to do that which, if not supported, will fail the request (i.e. one should not configure local computation if it cannot be done). Otherwise, we fall through to check if a script is available to do this (line 400) and if there is one it is invoked. So, if you specified that the checksum is to be computed using a script (or program) then the whole thing works regardless of whether or not extended attributes are supported. If you said you wanted local computation then the default implementation requires that extended attributes be supported, I suspect that the notion was that local computation should happen no matter what. The answer is that if you supplied a checksum manager plug-in then that might be possible (DPM does this). Otherwise, it's a no-no. I agree that the documentation should be clearer on this point. That said, it's hard to cover all the cases since there are way too many implicit variables that control whether or not a checksum can be computed at all."}],"action":{"name":"View Pull Request","url":"https://github.com/xrootd/xrootd/pull/769#issuecomment-406770740"}}}
[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/xrootd/xrootd/pull/769#issuecomment-406770740",
"url": "https://github.com/xrootd/xrootd/pull/769#issuecomment-406770740",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
},
{
"@type": "MessageCard",
"@context": "http://schema.org/extensions",
"hideOriginalBody": "false",
"originator": "AF6C5A86-E920-430C-9C59-A73278B5EFEB",
"title": "Re: [xrootd/xrootd] [XrdHTTP] Implement RFC3230 for providing resource digest (#769)",
"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\": 769,\n\"IssueComment\": \"{{IssueComment.value}}\"\n}"
}
]
},
{
"name": "Close pull request",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"PullRequestClose\",\n\"repositoryFullName\": \"xrootd/xrootd\",\n\"pullRequestId\": 769\n}"
},
{
"targets": [
{
"os": "default",
"uri": "https://github.com/xrootd/xrootd/pull/769#issuecomment-406770740"
}
],
"@type": "OpenUri",
"name": "View on GitHub"
},
{
"name": "Unsubscribe",
"@type": "HttpPOST",
"target": "https://api.github.com",
"body": "{\n\"commandName\": \"MuteNotification\",\n\"threadId\": 357396191\n}"
}
],
"themeColor": "26292E"
}
]
Use REPLY-ALL to reply to list
To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1