GitHub PullRequestReviewCommentEvent at 2014-05-25 17:28:09
Repository: ECALELF
Actor: hengne
Payload: {"comment":{"url":"https://api.github.com/repos/ECALELFS/ECALELF/pulls/comments/13031784","id":13031784,"diff_hunk":"@@ -189,6 +202,16 @@ if [ \"$RUNRANGE\" == \"allRange\" -o \"`echo $RUNRANGE |grep -c -P '[0-9]+-[0-9]+'`\"\n     unset RUNRANGE\n fi\n \n+# make argument.xml file if do MC\n+if [ \"$TYPE\" == \"ALCARECOSIM\" ] && [ -n \"${CREATE}\" ];then","path":"ALCARAW_RECO/scripts/prodAlcareco.sh","position":64,"original_position":64,"commit_id":"063b6f19ef142168bdbcf2da519df82c9ba33b04","original_commit_id":"063b6f19ef142168bdbcf2da519df82c9ba33b04","user":{"login":"hengne","id":4972340,"avatar_url":"https://avatars.githubusercontent.com/u/4972340?","gravatar_id":"477689e633a690e6c6e70dba42fe4b10","url":"https://api.github.com/users/hengne","html_url":"https://github.com/hengne","followers_url":"https://api.github.com/users/hengne/followers","following_url":"https://api.github.com/users/hengne/following{/other_user}","gists_url":"https://api.github.com/users/hengne/gists{/gist_id}","starred_url":"https://api.github.com/users/hengne/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hengne/subscriptions","organizations_url":"https://api.github.com/users/hengne/orgs","repos_url":"https://api.github.com/users/hengne/repos","events_url":"https://api.github.com/users/hengne/events{/privacy}","received_events_url":"https://api.github.com/users/hengne/received_events","type":"User","site_admin":false},"body":"I\r\n\r\nOn Sun, May 25, 2014 at 7:13 PM, shervin86 <notifications@github.com> wrote:\r\n\r\n> In ALCARAW_RECO/scripts/prodAlcareco.sh:\r\n>\r\n> > @@ -189,6 +202,16 @@ if [ \"$RUNRANGE\" == \"allRange\" -o \"`echo $RUNRANGE |grep -c -P '[0-9]+-[0-9]+'`\"\r\n> >      unset RUNRANGE\r\n> >  fi\r\n> >\r\n> > +# make argument.xml file if do MC\r\n> > +if [ \"$TYPE\" == \"ALCARECOSIM\" ] && [ -n \"${CREATE}\" ];then\r\n>\r\n> As discussed on Thursday, it's not acceptable to run jobs at CAF using\r\n> data from other storage elements.\r\n> It violates the GRID principle. Data should be published and transferred\r\n> with Phedex to the T2_CH_CERN.\r\n>\r\nI don't agree. The Xrootd service provides the functionality to access a\r\nwide range of sites. Check out the wikipage below:\r\n\r\nhttps://twiki.cern.ch/twiki/bin/view/CMSPublic/WorkBookXrootdService#UseCrab\r\n\r\nAnd I don't see a \"grid rule\" that say it is \"not acceptable\".  It is at\r\nmost \"not suggested\". I wouldn't simply through the effort of developing\r\nthis xrootd into trash can by saying it is \"not acceptable\" or \"violates\r\nthe grid principle\".  It can solve big and urgent problems in e.g. a\r\nphysics analysis if the data transfer by Phedex is slow, or be refused\r\n(e.g. in my case).\r\n\r\n\r\n\r\n\r\n\r\n> —\r\n> Reply to this email directly or view it on GitHub<https://github.com/ECALELFS/ECALELF/pull/21/files#r13031728>\r\n> .\r\n>","created_at":"2014-05-25T17:28:09Z","updated_at":"2014-05-25T17:28:09Z","html_url":"https://github.com/ECALELFS/ECALELF/pull/21#discussion_r13031784","pull_request_url":"https://api.github.com/repos/ECALELFS/ECALELF/pulls/21","_links":{"self":{"href":"https://api.github.com/repos/ECALELFS/ECALELF/pulls/comments/13031784"},"html":{"href":"https://github.com/ECALELFS/ECALELF/pull/21#discussion_r13031784"},"pull_request":{"href":"https://api.github.com/repos/ECALELFS/ECALELF/pulls/21"}}}}