{"action":"created","issue":{"url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/issues/102","repository_url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3","labels_url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/issues/102/labels{/name}","comments_url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/issues/102/comments","events_url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/issues/102/events","html_url":"https://github.com/logstash-plugins/logstash-output-s3/pull/102","id":177344743,"number":102,"title":"Refactoring, Dynamic prefix and AWS v2","user":{"login":"ph","id":640,"avatar_url":"https://avatars.githubusercontent.com/u/640?v=3","gravatar_id":"","url":"https://api.github.com/users/ph","html_url":"https://github.com/ph","followers_url":"https://api.github.com/users/ph/followers","following_url":"https://api.github.com/users/ph/following{/other_user}","gists_url":"https://api.github.com/users/ph/gists{/gist_id}","starred_url":"https://api.github.com/users/ph/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ph/subscriptions","organizations_url":"https://api.github.com/users/ph/orgs","repos_url":"https://api.github.com/users/ph/repos","events_url":"https://api.github.com/users/ph/events{/privacy}","received_events_url":"https://api.github.com/users/ph/received_events","type":"User","site_admin":false},"labels":[{"id":139594515,"url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/labels/enhancement","name":"enhancement","color":"009800","default":true},{"id":227233493,"url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/labels/needs%20reviewing","name":"needs reviewing","color":"cc317c","default":false}],"state":"open","locked":false,"assignee":{"login":"andrewvc","id":131427,"avatar_url":"https://avatars.githubusercontent.com/u/131427?v=3","gravatar_id":"","url":"https://api.github.com/users/andrewvc","html_url":"https://github.com/andrewvc","followers_url":"https://api.github.com/users/andrewvc/followers","following_url":"https://api.github.com/users/andrewvc/following{/other_user}","gists_url":"https://api.github.com/users/andrewvc/gists{/gist_id}","starred_url":"https://api.github.com/users/andrewvc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrewvc/subscriptions","organizations_url":"https://api.github.com/users/andrewvc/orgs","repos_url":"https://api.github.com/users/andrewvc/repos","events_url":"https://api.github.com/users/andrewvc/events{/privacy}","received_events_url":"https://api.github.com/users/andrewvc/received_events","type":"User","site_admin":false},"assignees":[{"login":"andrewvc","id":131427,"avatar_url":"https://avatars.githubusercontent.com/u/131427?v=3","gravatar_id":"","url":"https://api.github.com/users/andrewvc","html_url":"https://github.com/andrewvc","followers_url":"https://api.github.com/users/andrewvc/followers","following_url":"https://api.github.com/users/andrewvc/following{/other_user}","gists_url":"https://api.github.com/users/andrewvc/gists{/gist_id}","starred_url":"https://api.github.com/users/andrewvc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrewvc/subscriptions","organizations_url":"https://api.github.com/users/andrewvc/orgs","repos_url":"https://api.github.com/users/andrewvc/repos","events_url":"https://api.github.com/users/andrewvc/events{/privacy}","received_events_url":"https://api.github.com/users/andrewvc/received_events","type":"User","site_admin":false}],"milestone":null,"comments":24,"created_at":"2016-09-16T04:50:50Z","updated_at":"2016-12-08T17:49:12Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/pulls/102","html_url":"https://github.com/logstash-plugins/logstash-output-s3/pull/102","diff_url":"https://github.com/logstash-plugins/logstash-output-s3/pull/102.diff","patch_url":"https://github.com/logstash-plugins/logstash-output-s3/pull/102.patch"},"body":"**Motivation**\nOne of the most requested features was adding a way to add dynamic prefixes using the fieldref\nsyntax for the files on the bucket and also the changes in the pipeline to support shared delegator.\nThe S3 output by nature was always a single threaded writes but had multiples workers to process the upload, the code was threadsafe when used in the concurrency `:single` mode.\n\nThis PR addresses a few problems and provide shorter and more structured code:\n- This Plugin now uses the V2 version of the SDK, this make sure we receive the latest updates and changes.\n- We now uses S3's `upload_file` instead of reading chunks, this method is more efficient and will uses the multipart with threads if the files is too big.\n- You can now use the `fieldref` syntax in the prefix to dynamically changes the target with the events it receives.\n- The Upload queue is now a bounded list, this options is necessary to allow back pressure to be communicated back to the pipeline but its configurable by the user.\n- If the queue is full the plugin will start the upload in the current thread.\n- The plugin now threadsafe and support the concurrency model `shared`\n- The rotation strategy can be selected, the recommended is `size_and_time` that will check for both the configured limits (`size` and `time` are also available)\n- The `restore` option will now use a separate threadpool with an unbounded queue\n- The `restore` option will not block the launch of logstash and will uses less resources than the real time path\n- The plugin now uses `multi_receive_encode`, this will optimize the writes to the files\n- rotate operation are now batched to reduce the number of IO calls.\n- Empty file will not be uploaded by any rotation rotation strategy\n- We now use Concurrent-Ruby for the implementation of the java executor\n- If you have finer grain permission on prefixes or want faster boot, you can disable the credentials check with `validate_credentials_on_root_bucket`\n- The credentials check will no longer fails if we can't delete the file\n- We now have a full suite of integration test for all the defined rotation\n\nFixes: #4 #81 #44 #59 #50\n"},"comment":{"url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/issues/comments/265806136","html_url":"https://github.com/logstash-plugins/logstash-output-s3/pull/102#issuecomment-265806136","issue_url":"https://api.github.com/repos/logstash-plugins/logstash-output-s3/issues/102","id":265806136,"user":{"login":"andrewvc","id":131427,"avatar_url":"https://avatars.githubusercontent.com/u/131427?v=3","gravatar_id":"","url":"https://api.github.com/users/andrewvc","html_url":"https://github.com/andrewvc","followers_url":"https://api.github.com/users/andrewvc/followers","following_url":"https://api.github.com/users/andrewvc/following{/other_user}","gists_url":"https://api.github.com/users/andrewvc/gists{/gist_id}","starred_url":"https://api.github.com/users/andrewvc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andrewvc/subscriptions","organizations_url":"https://api.github.com/users/andrewvc/orgs","repos_url":"https://api.github.com/users/andrewvc/repos","events_url":"https://api.github.com/users/andrewvc/events{/privacy}","received_events_url":"https://api.github.com/users/andrewvc/received_events","type":"User","site_admin":false},"created_at":"2016-12-08T17:49:12Z","updated_at":"2016-12-08T17:49:12Z","body":"Left a few more comments, it's almost good to go!"}}
{"actor":{"display_login":"andrewvc"}}