Riverside-Software / sonar-openedge

CABL (Code Analyzer for ABL in SonarQube) - ABL ANTLR4 Parser
https://riverside-software.fr
GNU Lesser General Public License v3.0
62 stars 25 forks source link

dblist.txt in .sonarlint folder forgets databases #1047

Open dominickdm opened 1 year ago

dominickdm commented 1 year ago

Hello,

Regarding SonarLint - eclipse plugin: sometimes our .schema files got corrupted, is that a known issue? Every +- 2 weeks I have a colleague with parse errors due to tables that are not found anymore.

SonarLint processing file XXXX Found 0 issue(s) Parser error -- XXX -- Check analysis log for details

The fix is easy, we remove all files in the .SonarLint folder and the files are recreated again when a file is sonar-linted.

Do you see an improvement to either:

gquerret commented 1 year ago

Apologies for the delay. There are no known issues with the .schema files (but clearly doesn't mean it's bug-free). Next time you have an issue, could you send me the content of the .schema file (by email if you don't want it to appear here). I'll have a look at potential causes in the meantime.

dominickdm commented 1 year ago

Hello Gilles, We had the issue again. The schema files are ok, but it seems that the dblist.txt forgot 2 of the 3 databases that are connected.

If needed, I can send you the files.

gquerret commented 1 year ago

It is likely to come from a synchronization issue, such as analysis triggered while the AVM session was not connected to all sessions, or one DB was manually disconnected. It's possible to compare the number of DB in the cache to the expected value in the project config, and recreate the cache if the value is different. I'll see if there's some impact. Otherwise, the cache is always recreated every 24 hours.

ccecvb commented 1 year ago

FWIW I logged a case that PDSOE should not consider the AVM as started when the databases are not yet connected. It's logged as a defect.

Defect # OCTA-54598 Public Description PDSOE considers AVM start successful when no database is connected