Minutes: JDO TCK Conference Call Thursday April 20 1100 PDT 2000 CEST

2023-04-20 Thread Craig Russell
Attendees: Michael Bouschen, Tilmann Zäschke, Tobias Bouschen, Craig Russell

Next meeting: Wednesday April 26 1200 PDT 2100 CEST

Agenda:

1. sonarcloud issues

JIRA JDO-819 "Code quality analysis" 
https://issues.apache.org/jira/browse/JDO-819
  JIRA JDO-823 "Fix sonarcloud issues of type Code Smells" 
https://issues.apache.org/jira/browse/JDO-823
See PR #75 "JDO-823 - Make JDOException immutable" 
https://github.com/apache/db-jdo/pull/75

The current behavior is better than the suggested fix. 
Mark printStackTrace as Won't Fix.
Fix the non-final nested and failed.
AI Tobias: new PR to fix the two.

  Sonarcloud link: https://sonarcloud.io/summary/overall?id=db-jdo

* Cognitive Complexity of methods should not be too high:
  
https://sonarcloud.io/project/issues?resolved=false=java%3AS3776=CRITICAL=CODE_SMELL=db-jdo

Volunteers needed.

* Raw types should not be used:
  
https://sonarcloud.io/project/issues?resolved=false=java%3AS3740=MAJOR=db-jdo

Breaking change; keep on the agenda.

2. field/property converters

JIRA JDO-709 "Standardize field/property converters" 
https://issues.apache.org/jira/browse/JDO-709
JIRA JDO-826 " @Convert Javadoc indicates support for 
@Target(ElementType.TYPE,...)" https://issues.apache.org/jira/browse/JDO-826
See PR #74 "JDO 826 - Fix @Convert javadoc for types" 
https://github.com/apache/db-jdo/pull/74

Good to go.
This makes it clear that we are not looking at changing it.

3. JIRA JDO-822: "Verify compatibility with JDK 20" 
https://issues.apache.org/jira/browse/JDO-822

4. JIRA JDO-812 "Move to JDK 11 as the lowest supported version" 
https://issues.apache.org/jira/browse/JDO-812

5. Other issues

Action Items from weeks past:
[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 
datastores" https://issues.apache.org/jira/browse/JDO-651

Craig L Russell
c...@apache.org



[GitHub] [db-jdo] sonarcloud[bot] commented on pull request #76: JDO-823 - Make fields in JDOException final

2023-04-20 Thread via GitHub


sonarcloud[bot] commented on PR #76:
URL: https://github.com/apache/db-jdo/pull/76#issuecomment-1516777040

   Kudos, SonarCloud Quality Gate passed!  [![Quality Gate 
passed](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/QualityGateBadge/passed-16px.png
 'Quality Gate 
passed')](https://sonarcloud.io/dashboard?id=db-jdo=76)
   
   
[![Bug](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/bug-16px.png
 
'Bug')](https://sonarcloud.io/project/issues?id=db-jdo=76=false=BUG)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=db-jdo=76=false=BUG)
 [0 
Bugs](https://sonarcloud.io/project/issues?id=db-jdo=76=false=BUG)
  
   
[![Vulnerability](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/vulnerability-16px.png
 
'Vulnerability')](https://sonarcloud.io/project/issues?id=db-jdo=76=false=VULNERABILITY)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=db-jdo=76=false=VULNERABILITY)
 [0 
Vulnerabilities](https://sonarcloud.io/project/issues?id=db-jdo=76=false=VULNERABILITY)
  
   [![Security 
Hotspot](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/security_hotspot-16px.png
 'Security 
Hotspot')](https://sonarcloud.io/project/security_hotspots?id=db-jdo=76=false=SECURITY_HOTSPOT)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/security_hotspots?id=db-jdo=76=false=SECURITY_HOTSPOT)
 [0 Security 
Hotspots](https://sonarcloud.io/project/security_hotspots?id=db-jdo=76=false=SECURITY_HOTSPOT)
  
   [![Code 
Smell](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/common/code_smell-16px.png
 'Code 
Smell')](https://sonarcloud.io/project/issues?id=db-jdo=76=false=CODE_SMELL)
 
[![A](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/RatingBadge/A-16px.png
 
'A')](https://sonarcloud.io/project/issues?id=db-jdo=76=false=CODE_SMELL)
 [0 Code 
Smells](https://sonarcloud.io/project/issues?id=db-jdo=76=false=CODE_SMELL)
   
   [![No Coverage 
information](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/CoverageChart/NoCoverageInfo-16px.png
 'No Coverage 
information')](https://sonarcloud.io/component_measures?id=db-jdo=76)
 No Coverage information  
   
[![0.0%](https://sonarsource.github.io/sonarcloud-github-static-resources/v2/checks/Duplications/3-16px.png
 
'0.0%')](https://sonarcloud.io/component_measures?id=db-jdo=76=new_duplicated_lines_density=list)
 [0.0% 
Duplication](https://sonarcloud.io/component_measures?id=db-jdo=76=new_duplicated_lines_density=list)
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jdo-dev-unsubscr...@db.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [db-jdo] tobous opened a new pull request, #76: JDO-823 - Make fields in JDOException final

2023-04-20 Thread via GitHub


tobous opened a new pull request, #76:
URL: https://github.com/apache/db-jdo/pull/76

   Make all fields in JDOException final. Explicitly excludes 
'inPrintStackTrace' as it is needed for some state tracking. Makes it 
'transient' instead.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jdo-dev-unsubscr...@db.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [db-jdo] tzaeschke merged pull request #74: JDO 826 - Fix @Convert javadoc for types

2023-04-20 Thread via GitHub


tzaeschke merged PR #74:
URL: https://github.com/apache/db-jdo/pull/74


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jdo-dev-unsubscr...@db.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [db-jdo] tobous commented on pull request #75: JDO-823 - Make JDOException immutable

2023-04-20 Thread via GitHub


tobous commented on PR #75:
URL: https://github.com/apache/db-jdo/pull/75#issuecomment-1516770471

   Rejected. Old behavior is preferred. `JDOException` being mutable is too low 
of an issue for the larger rework necessary to replicate the old behavior while 
keeping the class immutable.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jdo-dev-unsubscr...@db.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[GitHub] [db-jdo] tobous closed pull request #75: JDO-823 - Make JDOException immutable

2023-04-20 Thread via GitHub


tobous closed pull request #75: JDO-823 - Make JDOException immutable
URL: https://github.com/apache/db-jdo/pull/75


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: jdo-dev-unsubscr...@db.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org