Make sense. +1 Thks Amol
E:a...@datatorrent.com | M: 510-449-2606 | Twitter: @*amolhkekre* www.datatorrent.com On Fri, Sep 8, 2017 at 9:16 AM, Vlad Rozov <v.rozo...@gmail.com> wrote: > Committers, > > Please make sure to follow Apex community guideline when merging PR > http://apex.apache.org/contributing.html. > > 1. Ensure that basic requirements for a pull request are met. This > includes: > * Sufficient time has passed for others to review > * PR was suffiently reviewed and comments were addressed. > Seevoting policy <https://www.apache.org/foundation/voting.html>. > * When there are multiple reviewers, wait till other reviewers > approve, with timeout of 48 hours before merging > * /If the PR was open for a long time, email dev@ declaring intent > to merge/ > * Commit messages and PR title need to reference JIRA (pull > requests will be linked to ticket) > * /Travis CI and Jenkins pull request build needs to pass/ > * /Ensure tests are added/modified for new features or fixes/ > * Ensure appropriate JavaDoc comments have been added > * Verify contributions don't depend on incompatible licences > (seehttps://www.apache.org/legal/resolved.html#category-x) > 2. Use the github/rebase and merge/option or the git command line to > merge the pull request (see link|view command line options|on the PR). > 3. /Update JIRA after pushing the changes. Set the////|Fix > version|////field and resolve the JIRA with proper resolution. *Also > verify that other fields (type, priority, assignee) are correct*./ > > > A couple of recent PR merges (#661, #669) to apex-malhar require a second > look from the committers. > > Thank you, > > Vlad >