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 8705 extensions #6672

Closed
wants to merge 11 commits into from

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 Jul 9, 2021

This pull request introduces 1 alert when merging d0310ef into 2c502d0 - view on LGTM.com

new alerts:

  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Jul 12, 2021

This pull request introduces 1 alert when merging 31e2e56 into f0e328b - view on LGTM.com

new alerts:

  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Jul 12, 2021

This pull request introduces 2 alerts when merging 42b98ee into f0e328b - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2021

This pull request introduces 2 alerts when merging b37d9f3 into 90bc5b7 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 13, 2021

This pull request introduces 2 alerts when merging 1f79862 into 90bc5b7 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2021

This pull request introduces 2 alerts when merging e494397 into e3457ff - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 15, 2021

This pull request introduces 2 alerts when merging c714c41 into e3457ff - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 16, 2021

This pull request introduces 2 alerts when merging 9043dd6 into 9704979 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 16, 2021

This pull request introduces 2 alerts when merging 99ada23 into 9704979 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 16, 2021

This pull request introduces 2 alerts when merging d7f9577 into 9704979 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 19, 2021

This pull request introduces 2 alerts when merging e795a61 into 4618604 - view on LGTM.com

new alerts:

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

@kohlmu-pivotal kohlmu-pivotal force-pushed the GEODE-8705-extensions branch 2 times, most recently from 1df5ad2 to d9de048 Compare July 21, 2021 03:38
@lgtm-com
Copy link

lgtm-com bot commented Jul 21, 2021

This pull request introduces 2 alerts when merging d9de048 into d77cb31 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 22, 2021

This pull request introduces 2 alerts when merging 1f787f1 into 4338015 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 22, 2021

This pull request introduces 2 alerts when merging 463797d into d615470 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 23, 2021

This pull request introduces 2 alerts when merging 8f89813 into d615470 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 23, 2021

This pull request introduces 2 alerts when merging dcc4fa0 into d615470 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 23, 2021

This pull request introduces 2 alerts when merging 0431c16 into d615470 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 23, 2021

This pull request introduces 2 alerts when merging e4d4f74 into 4e70bfc - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 24, 2021

This pull request introduces 2 alerts when merging 320d495 into 4e70bfc - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 26, 2021

This pull request introduces 2 alerts when merging cd15156 into 4e70bfc - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 27, 2021

This pull request introduces 2 alerts when merging d6c1b05 into 7d08ab5 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Jul 27, 2021

This pull request introduces 2 alerts when merging b9bd6be into 7e246f5 - view on LGTM.com

new alerts:

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

@lgtm-com
Copy link

lgtm-com bot commented Oct 5, 2021

This pull request introduces 4 alerts when merging 7943358 into b73065f - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Oct 7, 2021

This pull request introduces 4 alerts when merging 7fd03b7 into fb93bea - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Oct 11, 2021

This pull request introduces 4 alerts when merging 272f94a into 518e213 - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Oct 12, 2021

This pull request introduces 4 alerts when merging d64a8c4 into 4ae106f - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@kohlmu-pivotal kohlmu-pivotal force-pushed the GEODE-8705-extensions branch 2 times, most recently from 05a0768 to cc7a267 Compare October 13, 2021 20:19
@lgtm-com
Copy link

lgtm-com bot commented Oct 13, 2021

This pull request introduces 4 alerts when merging cc7a267 into 615f180 - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Oct 14, 2021

This pull request introduces 4 alerts when merging bedc80d into 567460f - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2021

This pull request introduces 4 alerts when merging 11a06bf into 636bea3 - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Oct 19, 2021

This pull request introduces 4 alerts when merging 4749cdb into a2a80ed - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

yozaner1324 and others added 8 commits October 21, 2021 11:39
* GEODE-8905: Introduce JarDeploymentService.
   Unified deploy functionality between REST and GFSH
   Pulled deploy jar functionality into its own module
   Renamed internal *Util classes to *Utils

Co-authored-by: Udo Kohlmeyer <ukohlmeyer@pivotal.io>
	   - Add ModularClasspathService
	   - Add testcontainers for testing in acceptance test
           - Add verification when deploying jars with the "geode-" in the path.

GEODE-8705 - Adding ability to run DUnits with modularity enabled
@lgtm-com
Copy link

lgtm-com bot commented Oct 21, 2021

This pull request introduces 4 alerts when merging 63e1954 into 9079de9 - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Oct 21, 2021

This pull request introduces 4 alerts when merging 4c8ebe2 into 9079de9 - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@lgtm-com
Copy link

lgtm-com bot commented Oct 21, 2021

This pull request introduces 4 alerts when merging feff9d2 into 9079de9 - view on LGTM.com

new alerts:

  • 2 for Container contents are never accessed
  • 1 for Use of a broken or risky cryptographic algorithm
  • 1 for Potential input resource leak

@yozaner1324 yozaner1324 closed this Feb 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant