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/06/17 20:55:42 UTC
[GitHub] [geode] mivanac opened a new pull request #5270: GEODE-6150: added test
mivanac opened a new pull request #5270:
URL: https://github.com/apache/geode/pull/5270
WIP: Do not review
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] onichols-pivotal commented on pull request #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
onichols-pivotal commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-700945786
this PR appears to have been abandoned
----------------------------------------------------------------
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] mivanac commented on pull request #5270: GEODE-6150: added test (Allow use of client/server max-threads with SSL)
Posted by GitBox <gi...@apache.org>.
mivanac commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-787123996
New PR created
----------------------------------------------------------------
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] mivanac closed pull request #5270: GEODE-6150: added test (Allow use of client/server max-threads with SSL)
Posted by GitBox <gi...@apache.org>.
mivanac closed pull request #5270:
URL: https://github.com/apache/geode/pull/5270
----------------------------------------------------------------
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 #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-646804201
This pull request **introduces 9 alerts** and **fixes 2** when merging 9a7df461f8c54fed8fb8d56ac32fce0155eb6ed4 into 9939cc0f2f1caad051bd104a0a06a4e1737d3830 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-ecec6003f1a0a6c9c1e7f748d31cc08765efaedc)
**new alerts:**
* 4 for Dereferenced variable may be null
* 3 for Potential output resource leak
* 2 for Potential input resource leak
**fixed alerts:**
* 2 for Potential input resource leak
----------------------------------------------------------------
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 #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-715586038
This pull request **introduces 10 alerts** and **fixes 2** when merging 0d64a52b557f693d2efeb32183e6250e884abdf5 into 0fdfc1ad1e272eed9bf5e2ee59816b75b262d9c6 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-370dabfbf4f68f1f62b81c97f126c51f64625bd7)
**new alerts:**
* 4 for Potential output resource leak
* 4 for Dereferenced variable may be null
* 2 for Potential input resource leak
**fixed alerts:**
* 2 for Potential input resource leak
----------------------------------------------------------------
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 commented on pull request #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
kohlmu-pivotal commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-716267225
@mivanac could you please have a look at this PR again.
Please either close or update for review.
----------------------------------------------------------------
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 #5270: GEODE-6150: added test (Allow use of client/server max-threads with SSL)
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-779888021
This pull request **fixes 2 alerts** when merging 6339caeb6c8ce2307cb9b08707299fd34df9a5d0 into ea37a2d31e023aa5645c6404d1d55191e689b926 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-f9de1a3759e35d42ef8a39786d7d2d488d5ac10f)
**fixed alerts:**
* 2 for Potential input resource leak
----------------------------------------------------------------
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 #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-715955682
This pull request **fixes 2 alerts** when merging 63821bfc619f9f0b59c1bc609388ccfaac509b83 into 0fdfc1ad1e272eed9bf5e2ee59816b75b262d9c6 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-4b2bc41f812441a97b92b4283e0d76ca08d1f54d)
**fixed alerts:**
* 2 for Potential input resource leak
----------------------------------------------------------------
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 #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-715887878
This pull request **introduces 2 alerts** and **fixes 2** when merging 45ea842b076572d7c25f0bf6d86da8ae2d011295 into 0fdfc1ad1e272eed9bf5e2ee59816b75b262d9c6 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-209ec5380b6616a64566771776aa8597de9a6dd3)
**new alerts:**
* 2 for Dereferenced variable may be null
**fixed alerts:**
* 2 for Potential input resource leak
----------------------------------------------------------------
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 #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-645638238
This pull request **introduces 8 alerts** and **fixes 2** when merging 5ee88eb8a6ae6cf2113bfefd8d9d6f17dd6e3df9 into d1e857de0d32742f65768e246c49db88ba90cc47 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-0f8ed490f8c9de4d3f88b474712741c005d50a9b)
**new alerts:**
* 3 for Potential output resource leak
* 3 for Dereferenced variable may be null
* 2 for Potential input resource leak
**fixed alerts:**
* 2 for Potential input resource leak
----------------------------------------------------------------
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 #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-650218615
This pull request **introduces 9 alerts** and **fixes 2** when merging f5bd6884b10c1b5b100a678bb6ff1eba4b85ab18 into c9ed7d7d068cee0b8bb2f626e78a1c0090fad92d - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-0a2d97f5ef4d21f4b936c368de54429c89d6dee9)
**new alerts:**
* 4 for Dereferenced variable may be null
* 3 for Potential output resource leak
* 2 for Potential input resource leak
**fixed alerts:**
* 2 for Potential input resource leak
----------------------------------------------------------------
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 #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-649068114
----------------------------------------------------------------
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 #5270: GEODE-6150: added test
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5270:
URL: https://github.com/apache/geode/pull/5270#issuecomment-648800451
This pull request **introduces 9 alerts** and **fixes 2** when merging 43052f39ff8b21c33ed3c7c0b79cd9f28188c75f into ca5d961bf3e79e88d06405604b6eb25c2acfd238 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-d0b7822382feb1b1e01a58c8fd018eda8978105c)
**new alerts:**
* 4 for Dereferenced variable may be null
* 3 for Potential output resource leak
* 2 for Potential input resource leak
**fixed alerts:**
* 2 for Potential input resource leak
----------------------------------------------------------------
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