OK, I removed the NOSONAR comment from JDOException's printStackTrace methods.

I see the point of not cluttering the code with NOSONAR comments. Just for the discussion, there is one downside marking the issues in the web UI: these settings are lost when we would switch to another sonarcloud or sonarqube instance.

I doubt that sonarcloud is scanning the projects in oder to look for false positives. I think we have to report them explicitly, if we want to bring a false postive ti their attention.

In the case of the JDOException's printStackTrace methods I propose them to resolved them as won't fix. The analysis is correct, we use a parameter PrintStream for synchronization and in general this is not a good idea. But here we do not want the PrintStream to be used in parallel, so I think "Resolve as won't fix" is the better solution.

Regards Michael
Hi Til,

Marking the issue as false positive seems to be better than modifying the code.

I'm also thinking that SonarCloud might pay more attention to false positives 
than to NOSONAR in the code...


On Dec 16, 2022, at 08:05, Tilmann<tilmann_...@gmx.de>  wrote:

4. JIRA JDO-819 "Code quality analysis"

I forgot to mention, instead of '// NOSONAR' it is also easily possible
to mark problems as "false positive" in the web UI.

In my opinion this would be preferred over '// NOSONAR' because using
the web UI avoids cluttering the code with '// NOSONAR' statements.

Maybe something to discuss in the next meeting.



On 16.12.22 16:49, Craig Russell wrote:
Attendees: Michael Bouschen, Tilmann Zäschke, Tobias Bouschen, Craig Russell

Next meeting: Thursday December 29 1100 PST 2000 CET


1. Derby vulnerability

No rush to upgrade Derby from 10.14.2 (current JDO dependency for tck) to 
10.14.3 (fixed version). The tck does not use LDAP which is the attack vector.
AI Michael look into upgrading.

2. JIRA JDO-820: "Clean up copyright NOTICE and 
Ist there anything left?
Nothing left. Resolved.

Which Fix Version do we want to use?
Next version whatever that is. 3.2.2 and 3.3

3. JIRA JDO-821: "Fix sonarcloud issues of type 
See PR #65:https://github.com/apache/db-jdo/pull/65  

4. JIRA JDO-819 "Code quality analysis"https://issues.apache.org/jira/browse/JDO-819  

Changes in PR#65 can be merged.
Tilmann is looking at the SonarCloud "security" issues. Seem to be innocuous.

Still more SonarCloud issues (code smell) to address. For example:
https://mx.akquinet.de/link?id=BAgAAABb9OSn1vfIQcIAAAC_H51M3PPC4kGHhS08O7AZGvxbxYmbyeOsFbBnKCsrqTv4paC17qVYZrqMpOr5KRcrrXQDm3lvFiNo11bzQdKPJkf3JkXYgKZ07VxoMnoFxgXCqMzowt3YXZYZtR7mTakRySOgk4g1dunhTTHu1LxSaieq3ZPLZH9T8hqGwg-9ukA-A3_RDrGE5dAqZ-VE2_cTfXVjw10VqLFw189pA7MDRA3F8lZaVvf1TPTfTKrT-QxK5HBtc0tzxeUwCpGgBaZlZw2 <https://mx.akquinet.de/link?id=BAgAAABb9OSn1vfIQcIAAAC_H51M3PPC4kGHhS08O7AZGvxbxYmbyeOsFbBnKCsrqTv4paC17qVYZrqMpOr5KRcrrXQDm3lvFiNo11bzQdKPJkf3JkXYgKZ07VxoMnoFxgXCqMzowt3YXZYZtR7mTakRySOgk4g1dunhTTHu1LxSaieq3ZPLZH9T8hqGwg-9ukA-A3_RDrGE5dAqZ-VE2_cTfXVjw10VqLFw189pA7MDRA3F8lZaVvf1TPTfTKrT-QxK5HBtc0tzxeUwCpGgBaZlZw2 >

Every subclass of JDOUserException smells. Not really an issue for us.

Anyone who looks into a SonarCloud issue should add a comment to the JDO-819 
JIRA with the analysis and resolution and possibly a PR. At some point we may 
close the issue after the items are low enough importance.

5. JIRA JDO-709 "Standardize field/property converters" 
https://issues.apache.org/jira/browse/JDO-709 6. JIRA JDO-815 "Change headers on source files to use 
https:// instead of http:// "https://issues.apache.org/jira/browse/JDO-815  

Ready to resolve.
AI Craig resolve it.

7. JIRA JDO-822: "Verify compatibility with JDK 

8. JIRA JDO-812 "Move to JDK 11 as the lowest supported version" https://issues.apache.org/jira/browse/JDO-812 9. Other issues Action Items from weeks past: [Nov 23 2022] AI Tilmann see what else is needed to have the analysis integrated into GitHub repo. [Nov 23 2022] AI Tilmann follow up with Andy/DataNucleus for his advice on JDO-709. [Oct 20 2022] AI Craig update the JIRA JDO-709 to request a test case using annotations and results of the test. [Dec 09 2021] AI Craig: Try to contact all current/former participants in JDO development and see if and how they want to be recognized on the JDO and DB web sites.https://db.apache.org/whoweare.html [Oct 07 2021] AI Craig send a private message to all JSR-243 Expert Group members asking if they wish to continue. [Mar 25 2021] AI Craig: investigate "merging" papajdo and apache.clr accounts
[Oct 17 2014] AI Matthew any updates for "Modify specification to address NoSQL 

Craig L Russell

Craig L Russell

Reply via email to