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

Refactor/singleton 1 #13269

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

Conversation

chrisdutz
Copy link
Contributor

@chrisdutz chrisdutz commented Aug 22, 2024

Description

This branch should be used to track the progress of moving away from using singletons.

In a first step I wrote some rules and added a new profile "with-architectural-analysis" that spots Singletons and their usage and reports that.

In order to collect this data during the build, be sure to run a build with that profile enabled:

mvn -P with-architectural-analysis clean verify

When building with enabled profile, the JQAssistant plugin will automatically analyse the project and collect all data in a Neo4J database which is located in "target/jqassistant". After the build with

mvn -P with-architectural-analysis jqassistant:report 

A nice HTML report can be generated.
When running:

mvn -P with-architectural-analysis jqassistant:server

A Neo4j server is started alongside the Neo4J web-UI. The user can then use the browser to analyse all sort of different things in IoTDB using the Neo4J graph query language: Cypher.

chrisdutz and others added 30 commits August 17, 2023 11:31
@liyuheng55555
Copy link
Collaborator

Hi Chris, I fetched your branch, tried mvn -P with-architectural-analysis jqassistant:report, but cannot found the generated html file. Where is this "nice HTML report" located at?

@chrisdutz
Copy link
Contributor Author

it should be in "target/site/" something ...

@liyuheng55555
Copy link
Collaborator

@chrisdutz still cannot find it... Which package's target directory are you referring to? Besides, I currently only called mvn -P with-architectural-analysis jqassistant:report. Do I need to build the distribution or execute additional commands beyond this?

@chrisdutz
Copy link
Contributor Author

image

So ... these two steps are needed:

mvn -P with-architectural-analysis clean verify

To collect the data, then:

mvn -P with-architectural-analysis jqassistant:report 

To produce the report as seen in my screenshot above.

@liyuheng55555
Copy link
Collaborator

image image

@chrisdutz Thanks for the guide, now I find the site dir, and mvn -P with-architectural-analysis jqassistant:server is able to connect to the server. By the way, can you provide some example query for neo4j? Chatgpt give me something which seems not working:

MATCH (singleton:Type:Singleton)
RETURN singleton
ORDER BY singleton.fqn

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.

2 participants