{"action":"created","issue":{"url":"https://api.github.com/repos/apache/spark/issues/15206","repository_url":"https://api.github.com/repos/apache/spark","labels_url":"https://api.github.com/repos/apache/spark/issues/15206/labels{/name}","comments_url":"https://api.github.com/repos/apache/spark/issues/15206/comments","events_url":"https://api.github.com/repos/apache/spark/issues/15206/events","html_url":"https://github.com/apache/spark/pull/15206","id":178750119,"number":15206,"title":"[SPARK-17640][SQL]Avoid using -1 as the default batchId for FileStreamSource.FileEntry","user":{"login":"zsxwing","id":1000778,"avatar_url":"https://avatars.githubusercontent.com/u/1000778?v=3","gravatar_id":"","url":"https://api.github.com/users/zsxwing","html_url":"https://github.com/zsxwing","followers_url":"https://api.github.com/users/zsxwing/followers","following_url":"https://api.github.com/users/zsxwing/following{/other_user}","gists_url":"https://api.github.com/users/zsxwing/gists{/gist_id}","starred_url":"https://api.github.com/users/zsxwing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zsxwing/subscriptions","organizations_url":"https://api.github.com/users/zsxwing/orgs","repos_url":"https://api.github.com/users/zsxwing/repos","events_url":"https://api.github.com/users/zsxwing/events{/privacy}","received_events_url":"https://api.github.com/users/zsxwing/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-09-22T23:55:50Z","updated_at":"2016-09-23T00:02:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/spark/pulls/15206","html_url":"https://github.com/apache/spark/pull/15206","diff_url":"https://github.com/apache/spark/pull/15206.diff","patch_url":"https://github.com/apache/spark/pull/15206.patch"},"body":"## What changes were proposed in this pull request?\r\n\r\nAvoid using -1 as the default batchId for FileStreamSource.FileEntry so that we can make sure not writing any FileEntry(..., batchId = -1) into the log. This also avoids people misusing it in future (#15203 is an example).\r\n\r\n## How was this patch tested?\r\n\r\nJenkins."},"comment":{"url":"https://api.github.com/repos/apache/spark/issues/comments/249062614","html_url":"https://github.com/apache/spark/pull/15206#issuecomment-249062614","issue_url":"https://api.github.com/repos/apache/spark/issues/15206","id":249062614,"user":{"login":"zsxwing","id":1000778,"avatar_url":"https://avatars.githubusercontent.com/u/1000778?v=3","gravatar_id":"","url":"https://api.github.com/users/zsxwing","html_url":"https://github.com/zsxwing","followers_url":"https://api.github.com/users/zsxwing/followers","following_url":"https://api.github.com/users/zsxwing/following{/other_user}","gists_url":"https://api.github.com/users/zsxwing/gists{/gist_id}","starred_url":"https://api.github.com/users/zsxwing/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/zsxwing/subscriptions","organizations_url":"https://api.github.com/users/zsxwing/orgs","repos_url":"https://api.github.com/users/zsxwing/repos","events_url":"https://api.github.com/users/zsxwing/events{/privacy}","received_events_url":"https://api.github.com/users/zsxwing/received_events","type":"User","site_admin":false},"created_at":"2016-09-23T00:02:16Z","updated_at":"2016-09-23T00:02:16Z","body":"> @zsxwing Will this break streaming jobs if someone upgrades their Spark version, since they won't be able to deserialize the class correctly?\r\n\r\n@brkyvz No. This doesn't change the file format. `FileEntry(..., batchId = -1)` is only used in `SeenFilesMap` which is only in memory.\r\n\r\nBy the way, people cannot just upgrade the Spark version since `FileStreamSource`'s format has been changed in #13513."}}
{"actor":{"display_login":"zsxwing"}}