[GitHub] [db-jdo] tzaeschke commented on pull request #23: Jdo 709 spec

[GitBox <gi...@apache.org>]

tzaeschke commented on pull request #23:
URL: https://github.com/apache/db-jdo/pull/23#issuecomment-855283591


   Some comments:
   * Chapter 6: Would it make sense to relabel "Converted Classes" to "Convertible Classes" or "Convertible Types"? In my mind the converted type (a primitive, not a class) is stored in the datastore. The application/domain type is in my mind the un-converted type.
   * Chapter 14, page 8: Change "... subsequent execute method" to "subsequent execution" or "subsequently called execute method"?
   * Chapter 14, page 15: The last two change just appear to add an empty bullet?
   * Chapter 19.1.4, page 4: The first line of the javadoc comment is missing a linebreak after `/**`: `/** * The`
   * Chapter 19.1.5, page 5: The first line of the javadoc comment is missing a linebreak after `/**`: `/** * The`


-- 
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.

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