payload: {"action":"created","issue":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22599","repository_url":"https://api.github.com/repos/tensorflow/tensorflow","labels_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22599/labels{/name}","comments_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22599/comments","events_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22599/events","html_url":"https://github.com/tensorflow/tensorflow/pull/22599","id":364958980,"node_id":"MDExOlB1bGxSZXF1ZXN0MjE5MDIxOTk4","number":22599,"title":"Skip record reader for cloud file system","user":{"login":"yupbank","id":741544,"node_id":"MDQ6VXNlcjc0MTU0NA==","avatar_url":"https://avatars1.githubusercontent.com/u/741544?v=4","gravatar_id":"","url":"https://api.github.com/users/yupbank","html_url":"https://github.com/yupbank","followers_url":"https://api.github.com/users/yupbank/followers","following_url":"https://api.github.com/users/yupbank/following{/other_user}","gists_url":"https://api.github.com/users/yupbank/gists{/gist_id}","starred_url":"https://api.github.com/users/yupbank/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yupbank/subscriptions","organizations_url":"https://api.github.com/users/yupbank/orgs","repos_url":"https://api.github.com/users/yupbank/repos","events_url":"https://api.github.com/users/yupbank/events{/privacy}","received_events_url":"https://api.github.com/users/yupbank/received_events","type":"User","site_admin":false},"labels":[{"id":300136587,"node_id":"MDU6TGFiZWwzMDAxMzY1ODc=","url":"https://api.github.com/repos/tensorflow/tensorflow/labels/cla:%20yes","name":"cla: yes","color":"009800","default":false},{"id":1169364458,"node_id":"MDU6TGFiZWwxMTY5MzY0NDU4","url":"https://api.github.com/repos/tensorflow/tensorflow/labels/size:S","name":"size:S","color":"adafea","default":false}],"state":"closed","locked":false,"assignee":{"login":"rthadur","id":43972606,"node_id":"MDQ6VXNlcjQzOTcyNjA2","avatar_url":"https://avatars2.githubusercontent.com/u/43972606?v=4","gravatar_id":"","url":"https://api.github.com/users/rthadur","html_url":"https://github.com/rthadur","followers_url":"https://api.github.com/users/rthadur/followers","following_url":"https://api.github.com/users/rthadur/following{/other_user}","gists_url":"https://api.github.com/users/rthadur/gists{/gist_id}","starred_url":"https://api.github.com/users/rthadur/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rthadur/subscriptions","organizations_url":"https://api.github.com/users/rthadur/orgs","repos_url":"https://api.github.com/users/rthadur/repos","events_url":"https://api.github.com/users/rthadur/events{/privacy}","received_events_url":"https://api.github.com/users/rthadur/received_events","type":"User","site_admin":false},"assignees":[{"login":"rthadur","id":43972606,"node_id":"MDQ6VXNlcjQzOTcyNjA2","avatar_url":"https://avatars2.githubusercontent.com/u/43972606?v=4","gravatar_id":"","url":"https://api.github.com/users/rthadur","html_url":"https://github.com/rthadur","followers_url":"https://api.github.com/users/rthadur/followers","following_url":"https://api.github.com/users/rthadur/following{/other_user}","gists_url":"https://api.github.com/users/rthadur/gists{/gist_id}","starred_url":"https://api.github.com/users/rthadur/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rthadur/subscriptions","organizations_url":"https://api.github.com/users/rthadur/orgs","repos_url":"https://api.github.com/users/rthadur/repos","events_url":"https://api.github.com/users/rthadur/events{/privacy}","received_events_url":"https://api.github.com/users/rthadur/received_events","type":"User","site_admin":false}],"milestone":null,"comments":7,"created_at":"2018-09-28T16:28:44Z","updated_at":"2019-04-04T22:48:48Z","closed_at":"2019-04-04T22:39:36Z","author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/tensorflow/tensorflow/pulls/22599","html_url":"https://github.com/tensorflow/tensorflow/pull/22599","diff_url":"https://github.com/tensorflow/tensorflow/pull/22599.diff","patch_url":"https://github.com/tensorflow/tensorflow/pull/22599.patch"},"body":"Addressing this https://github.com/tensorflow/tensorflow/issues/22598\r\n\r\nThe behaviour of gcs(google cloud file storage) file system is different than local file system.\r\n\r\nas\r\n```\r\n tf.gfile.Open('local_empty.tfrecord').read(100000000) == ''\r\n```\r\nwhile \r\n```\r\ntf.gfile.Open('gcs_empty_file.tfrecord').read(1000000)=='Out[11]: \"<?xml version='1.0'\r\n encoding='UTF-8'?><Error>\r\n<Code>InvalidRange</Code>\r\n<Message>The requested range cannot be satisfied. \r\n</Message>\r\n<Details>bytes=some number</Details></Error>\"'\r\n```\r\nso when we use tf.data.TFRecordDataset to process the tfrecord in the gcs, there would be\r\nDataLossError\r\n"},"comment":{"url":"https://api.github.com/repos/tensorflow/tensorflow/issues/comments/480090685","html_url":"https://github.com/tensorflow/tensorflow/pull/22599#issuecomment-480090685","issue_url":"https://api.github.com/repos/tensorflow/tensorflow/issues/22599","id":480090685,"node_id":"MDEyOklzc3VlQ29tbWVudDQ4MDA5MDY4NQ==","user":{"login":"yupbank","id":741544,"node_id":"MDQ6VXNlcjc0MTU0NA==","avatar_url":"https://avatars1.githubusercontent.com/u/741544?v=4","gravatar_id":"","url":"https://api.github.com/users/yupbank","html_url":"https://github.com/yupbank","followers_url":"https://api.github.com/users/yupbank/followers","following_url":"https://api.github.com/users/yupbank/following{/other_user}","gists_url":"https://api.github.com/users/yupbank/gists{/gist_id}","starred_url":"https://api.github.com/users/yupbank/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/yupbank/subscriptions","organizations_url":"https://api.github.com/users/yupbank/orgs","repos_url":"https://api.github.com/users/yupbank/repos","events_url":"https://api.github.com/users/yupbank/events{/privacy}","received_events_url":"https://api.github.com/users/yupbank/received_events","type":"User","site_admin":false},"created_at":"2019-04-04T22:48:48Z","updated_at":"2019-04-04T22:48:48Z","author_association":"MEMBER","body":"make sense, thank you"}}
other: {"actor":{"display_login":"yupbank"}}