Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GEODE-8466: Introduce classloaderservice #5658

Closed

Conversation

yozaner1324
Copy link
Contributor

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?

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.

@lgtm-com
Copy link

lgtm-com bot commented Oct 23, 2020

This pull request introduces 3 alerts and fixes 1 when merging 9d12f2e into a048a72 - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@lgtm-com
Copy link

lgtm-com bot commented Oct 23, 2020

This pull request introduces 3 alerts and fixes 1 when merging dfc5bbc into 0fdfc1a - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@lgtm-com
Copy link

lgtm-com bot commented Oct 24, 2020

This pull request introduces 3 alerts and fixes 1 when merging 84ec38d into 0fdfc1a - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@kohlmu-pivotal kohlmu-pivotal changed the title Introduce classloaderservice GEODE-8466: Introduce classloaderservice Oct 27, 2020
@kohlmu-pivotal kohlmu-pivotal added this to Failing Pre-checkin in ClassLoader Isolation Oct 27, 2020
@lgtm-com
Copy link

lgtm-com bot commented Nov 3, 2020

This pull request introduces 3 alerts and fixes 1 when merging 30ece9b into 39db792 - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@lgtm-com
Copy link

lgtm-com bot commented Nov 4, 2020

This pull request introduces 3 alerts and fixes 1 when merging 2f94c29 into 39db792 - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@lgtm-com
Copy link

lgtm-com bot commented Nov 5, 2020

This pull request introduces 3 alerts and fixes 1 when merging 19b1313 into 7367d17 - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@lgtm-com
Copy link

lgtm-com bot commented Nov 6, 2020

This pull request introduces 3 alerts and fixes 1 when merging 43000b9 into af267c0 - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@lgtm-com
Copy link

lgtm-com bot commented Nov 7, 2020

This pull request introduces 3 alerts and fixes 1 when merging 640cd3f into c068979 - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@lgtm-com
Copy link

lgtm-com bot commented Nov 9, 2020

This pull request introduces 3 alerts and fixes 1 when merging 3bf7d27 into e1f5518 - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@lgtm-com
Copy link

lgtm-com bot commented Nov 10, 2020

This pull request introduces 3 alerts and fixes 1 when merging 846309e into e1f5518 - view on LGTM.com

new alerts:

  • 2 for Potential input resource leak
  • 1 for Use of a broken or risky cryptographic algorithm

fixed alerts:

  • 1 for Use of a broken or risky cryptographic algorithm

@yozaner1324 yozaner1324 closed this Jan 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
ClassLoader Isolation
Failing Pre-checkin
2 participants