You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2020/11/11 23:29:02 UTC

[GitHub] [geode] yozaner1324 opened a new pull request #5734: Geode 8461 - Add ModuleServerLauncher

yozaner1324 opened a new pull request #5734:
URL: https://github.com/apache/geode/pull/5734


   Thank you for submitting a contribution to Apache Geode.
   
   In order to streamline the review of the contribution we ask you
   to ensure the following steps have been taken:
   
   ### For all changes:
   - [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
   
   - [ ] Has your PR been rebased against the latest commit within the target branch (typically `develop`)?
   
   - [ ] Is your initial contribution a single, squashed commit?
   
   - [ ] Does `gradlew build` run cleanly?
   
   - [ ] Have you written or updated unit tests to verify your changes?
   
   - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?
   
   ### Note:
   Please ensure that once the PR is submitted, check Concourse for build issues and
   submit an update to your PR as soon as possible. If you need help, please send an
   email to dev@geode.apache.org.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] lgtm-com[bot] commented on pull request #5734: Geode 8461 - Add ModuleServerLauncher

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5734:
URL: https://github.com/apache/geode/pull/5734#issuecomment-726448679


   This pull request **introduces 2 alerts** and **fixes 2** when merging b062fbd37e28436fb9ea58ad5ffff986d8e5b97e into f23e01a32213245dc684b417d97a14cf5a026741 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-b4b10b072139d57b13afe55fb46f58f4284a6eb8)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] lgtm-com[bot] commented on pull request #5734: Geode 8461 - Add ModuleServerLauncher

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5734:
URL: https://github.com/apache/geode/pull/5734#issuecomment-726361130


   This pull request **introduces 2 alerts** and **fixes 2** when merging 48d0808eb6aa7e7456f0de17ec0b42dfc8d42b8b into f23e01a32213245dc684b417d97a14cf5a026741 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-c87efe45df95a725ffe5e54b955894c835cede72)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] lgtm-com[bot] commented on pull request #5734: Geode 8461 - Add ModuleServerLauncher

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5734:
URL: https://github.com/apache/geode/pull/5734#issuecomment-725734852


   This pull request **introduces 2 alerts** and **fixes 2** when merging 47f1d5fa89faf03d9ace6e79f7c0ecd78bc741d2 into c99087aeb19abfb5bbd57036349870a6d784df1a - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-fc1f162ebd07a031d6d13a450c6bff46add98fb5)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] onichols-pivotal commented on pull request #5734: Geode 8461 - Add ModuleServerLauncher

Posted by GitBox <gi...@apache.org>.
onichols-pivotal commented on pull request #5734:
URL: https://github.com/apache/geode/pull/5734#issuecomment-773029102


   This PR has been inactive for some while, can it be closed?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [geode] kohlmu-pivotal closed pull request #5734: Geode 8461 - Add ModuleServerLauncher

Posted by GitBox <gi...@apache.org>.
kohlmu-pivotal closed pull request #5734:
URL: https://github.com/apache/geode/pull/5734


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org