{"action":"created","issue":{"url":"https://api.github.com/repos/census-instrumentation/opencensus-proto/issues/56","repository_url":"https://api.github.com/repos/census-instrumentation/opencensus-proto","labels_url":"https://api.github.com/repos/census-instrumentation/opencensus-proto/issues/56/labels{/name}","comments_url":"https://api.github.com/repos/census-instrumentation/opencensus-proto/issues/56/comments","events_url":"https://api.github.com/repos/census-instrumentation/opencensus-proto/issues/56/events","html_url":"https://github.com/census-instrumentation/opencensus-proto/pull/56","id":324554966,"number":56,"title":"Metric: Wrap label values to distinguish between empty-string and unset.","user":{"login":"songy23","id":10536136,"avatar_url":"https://avatars3.githubusercontent.com/u/10536136?v=4","gravatar_id":"","url":"https://api.github.com/users/songy23","html_url":"https://github.com/songy23","followers_url":"https://api.github.com/users/songy23/followers","following_url":"https://api.github.com/users/songy23/following{/other_user}","gists_url":"https://api.github.com/users/songy23/gists{/gist_id}","starred_url":"https://api.github.com/users/songy23/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/songy23/subscriptions","organizations_url":"https://api.github.com/users/songy23/orgs","repos_url":"https://api.github.com/users/songy23/repos","events_url":"https://api.github.com/users/songy23/events{/privacy}","received_events_url":"https://api.github.com/users/songy23/received_events","type":"User","site_admin":false},"labels":[{"id":528873475,"url":"https://api.github.com/repos/census-instrumentation/opencensus-proto/labels/discussion","name":"discussion","color":"006b75","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":0,"created_at":"2018-05-18T21:17:42Z","updated_at":"2018-05-22T00:30:13Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/census-instrumentation/opencensus-proto/pulls/56","html_url":"https://github.com/census-instrumentation/opencensus-proto/pull/56","diff_url":"https://github.com/census-instrumentation/opencensus-proto/pull/56.diff","patch_url":"https://github.com/census-instrumentation/opencensus-proto/pull/56.patch"},"body":"We need a way to distinguish between empty-string and unset for label values. Since `label_values` is a repeated field, using the built-in `Wrapper.StringValue` doesn't help in this case. Instead we need to define our own `LabelValue` type that wraps the string value, as suggested by @bogdandrutu in https://github.com/census-instrumentation/opencensus-proto/pull/55#discussion_r189125133."},"comment":{"url":"https://api.github.com/repos/census-instrumentation/opencensus-proto/issues/comments/390825647","html_url":"https://github.com/census-instrumentation/opencensus-proto/pull/56#issuecomment-390825647","issue_url":"https://api.github.com/repos/census-instrumentation/opencensus-proto/issues/56","id":390825647,"user":{"login":"songy23","id":10536136,"avatar_url":"https://avatars3.githubusercontent.com/u/10536136?v=4","gravatar_id":"","url":"https://api.github.com/users/songy23","html_url":"https://github.com/songy23","followers_url":"https://api.github.com/users/songy23/followers","following_url":"https://api.github.com/users/songy23/following{/other_user}","gists_url":"https://api.github.com/users/songy23/gists{/gist_id}","starred_url":"https://api.github.com/users/songy23/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/songy23/subscriptions","organizations_url":"https://api.github.com/users/songy23/orgs","repos_url":"https://api.github.com/users/songy23/repos","events_url":"https://api.github.com/users/songy23/events{/privacy}","received_events_url":"https://api.github.com/users/songy23/received_events","type":"User","site_admin":false},"created_at":"2018-05-22T00:30:13Z","updated_at":"2018-05-22T00:30:13Z","author_association":"MEMBER","body":"Will merge this PR if there are no more comments/concerns by EOD tomorrow (PST)."}}
{"actor":{"display_login":"songy23"}}