Skip to content

Implement feature: detect redis by analyzing pom file#11

Merged
rujche merged 1 commit into
saragluna:java-analyzerfrom
rujche:rujche/java-analyzer
Oct 15, 2024
Merged

Implement feature: detect redis by analyzing pom file#11
rujche merged 1 commit into
saragluna:java-analyzerfrom
rujche:rujche/java-analyzer

Conversation

@rujche

@rujche rujche commented Oct 15, 2024

Copy link
Copy Markdown
Collaborator

Implement feature: detect redis by analyzing pom file.

1. Link to sample project.

https://github.com/rujche/samples/tree/azd-enhancemeng-for-redis

2. Make sure all redis related feature in spring-boot are included in this PR.

Search redis in this page: https://repo1.maven.org/maven2/org/springframework/boot/ We can got 4 artifacts:

  1. spring-boot-sample-data-redis-archetype. Ignore this because it's an archetype.
  2. spring-boot-starter-data-redis. Included in this PR.
  3. spring-boot-starter-data-redis-reactive. Included in this PR.
  4. spring-boot-starter-redis. Ignore this because the max version is 1.4.4, it's released on 2017-06-08)

Search redis in spring-configuration-metadata.json:

image

There are 4 groups:

  1. spring.cache.redis. Confirmed that the redis used in spring cache is using the RedisConnectionFactory configured by spring.data.redis.
  2. spring.data.redis. Included in the sample project.
  3. spring.session.redis. Confirmed that the redis used in spring session is using the RedisConnectionFactory configured by spring.data.redis.
  4. spring.redis. Confirmed that this has been replaced by spring.data.redis.

@rujche

rujche commented Oct 15, 2024

Copy link
Copy Markdown
Collaborator Author

Hi, @saragluna .
FYI: I'll this PR without your review.

@rujche rujche merged commit 8607043 into saragluna:java-analyzer Oct 15, 2024
saragluna pushed a commit that referenced this pull request Nov 11, 2024
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.

1 participant