{"action":"created","issue":{"url":"https://api.github.com/repos/openucx/ucx/issues/1421","repository_url":"https://api.github.com/repos/openucx/ucx","labels_url":"https://api.github.com/repos/openucx/ucx/issues/1421/labels{/name}","comments_url":"https://api.github.com/repos/openucx/ucx/issues/1421/comments","events_url":"https://api.github.com/repos/openucx/ucx/issues/1421/events","html_url":"https://github.com/openucx/ucx/pull/1421","id":219334266,"number":1421,"title":"UCT/API: Common struct for proto attrs","user":{"login":"brminich","id":14221180,"avatar_url":"https://avatars2.githubusercontent.com/u/14221180?v=3","gravatar_id":"","url":"https://api.github.com/users/brminich","html_url":"https://github.com/brminich","followers_url":"https://api.github.com/users/brminich/followers","following_url":"https://api.github.com/users/brminich/following{/other_user}","gists_url":"https://api.github.com/users/brminich/gists{/gist_id}","starred_url":"https://api.github.com/users/brminich/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/brminich/subscriptions","organizations_url":"https://api.github.com/users/brminich/orgs","repos_url":"https://api.github.com/users/brminich/repos","events_url":"https://api.github.com/users/brminich/events{/privacy}","received_events_url":"https://api.github.com/users/brminich/received_events","type":"User","site_admin":false},"labels":[{"id":284778078,"url":"https://api.github.com/repos/openucx/ucx/labels/api","name":"api","color":"5319e7","default":false},{"id":503059588,"url":"https://api.github.com/repos/openucx/ucx/labels/WIP-DNM","name":"WIP-DNM","color":"d93f0b","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-04-04T18:02:05Z","updated_at":"2017-04-05T19:06:48Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/openucx/ucx/pulls/1421","html_url":"https://github.com/openucx/ucx/pull/1421","diff_url":"https://github.com/openucx/ucx/pull/1421.diff","patch_url":"https://github.com/openucx/ucx/pull/1421.patch"},"body":"Common structure for protocol attributes may be useful, as it would allow abstraction of different operations (i. e. writing common functions for put, get, am, etc).\r\n\r\nNot all _uct_proto_att_r_ fields are required for every operation, so reducing the base attributes list would be an alternative (for instance _max_hdr_ is needed for AM only). But in this case more updates would be needed, because uct_iface_attr field names will be changed (for instance AM will need to combine basic _uct_proto_att_r_ struct and additional  _max_hdr_ field). \r\n\r\nNote: PR is created for discussion, doxygen comments are not correctly updated yet"},"comment":{"url":"https://api.github.com/repos/openucx/ucx/issues/comments/291964165","html_url":"https://github.com/openucx/ucx/pull/1421#issuecomment-291964165","issue_url":"https://api.github.com/repos/openucx/ucx/issues/1421","id":291964165,"user":{"login":"shamisp","id":2236120,"avatar_url":"https://avatars1.githubusercontent.com/u/2236120?v=3","gravatar_id":"","url":"https://api.github.com/users/shamisp","html_url":"https://github.com/shamisp","followers_url":"https://api.github.com/users/shamisp/followers","following_url":"https://api.github.com/users/shamisp/following{/other_user}","gists_url":"https://api.github.com/users/shamisp/gists{/gist_id}","starred_url":"https://api.github.com/users/shamisp/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/shamisp/subscriptions","organizations_url":"https://api.github.com/users/shamisp/orgs","repos_url":"https://api.github.com/users/shamisp/repos","events_url":"https://api.github.com/users/shamisp/events{/privacy}","received_events_url":"https://api.github.com/users/shamisp/received_events","type":"User","site_admin":false},"created_at":"2017-04-05T19:06:48Z","updated_at":"2017-04-05T19:06:48Z","body":"I think there was an agreement on the call that for now we do not have enough justification for the API change."}}
{"actor":{"display_login":"shamisp"}}