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

Polish #34

Merged
merged 1 commit into from
Mar 5, 2023
Merged

Polish #34

merged 1 commit into from
Mar 5, 2023

Conversation

chanhyeong
Copy link
Contributor

  • move duplicate synthetic patterns to the utility class
  • remove "throw" in orElseThrow()

@@ -7,7 +7,6 @@ include("scavenger-schema")
include("scavenger-agent-java")
include("scavenger-agent-python")
include("scavenger-model")
include("scavenger-old-agent-java")
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why?

Copy link
Contributor Author

@chanhyeong chanhyeong Mar 3, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm sorry. I turned back this line.
There was an error when building scavenger-old-agent-java in my device,
so I temporarily removed this line and ran test.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is the error message?
please tell me.. I think it's better to include old-agent-java conditionally.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are two kinds of errors depending on the JDK versions. (M1 Mac)

JDK 1.8

(azul, adoptium)

Build file '/Users/user/Documents/GitHub/scavenger/scavenger-old-agent-java/build.gradle.kts' line: 4

An exception occurred applying plugin request [id: 'com.palantir.git-version', version: '0.15.0']
> Failed to apply plugin 'com.palantir.git-version'.
   > Could not create plugin of type 'GitVersionPlugin'.
      > Could not generate a decorated class for type GitVersionPlugin.
         > org/eclipse/jgit/lib/Repository has been compiled by a more recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0

JDK 11, 17

(adoptium only)

Build file '/Users/user/Documents/GitHub/scavenger/scavenger-old-agent-java/build.gradle.kts' line: 112

No compatible toolchains found for request filter: {languageVersion=7, vendor=any, implementation=vendor-specific} (auto-detect true, auto-download true)

* Try:
> Run with --info or --debug option to get more log output.
> Run with --scan to get full insights.

* Exception is:
org.gradle.jvm.toolchain.internal.NoToolchainAvailableException: No compatible toolchains found for request filter: {languageVersion=7, vendor=any, implementation=vendor-specific} (auto-detect true, auto-download true)
	at org.gradle.jvm.toolchain.internal.JavaToolchainQueryService.lambda$noToolchainAvailable$5(JavaToolchainQueryService.java:96)
...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

An exception occurred applying plugin request [id: 'com.palantir.git-version', version: '0.15.0']

This error will be fixed by #39.

No compatible toolchains found for request filter: {languageVersion=7, vendor=any, implementation=vendor-specific} (auto-detect true, auto-download true)

This error will be resolved when you install JDK 7 on your machine. I assume this error is caused by gradle since Java 7 support is discontinued and gradle could not install it automatically (auto-download is enabled). I will track this issue on #38.

- move duplicate synthetic patterns to the utility class
- remove "throw" in orElseThrow()
@junoyoon
Copy link
Collaborator

junoyoon commented Mar 5, 2023

Merge this plz. @taeyeon-Kim

@taeyeon-Kim taeyeon-Kim merged commit 2b2dde3 into naver:develop Mar 5, 2023
@chanhyeong chanhyeong deleted the polish branch March 5, 2023 15:47
taeyeon-Kim added a commit that referenced this pull request Mar 6, 2023
Revert #34 because the agent and collector have different signatures

Reviewed-by: @kojandy
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants