ezoerner commented on code in PR #92:
URL: https://github.com/apache/db-jdo/pull/92#discussion_r1501693614
##########
api/src/main/java/javax/jdo/Query.java:
##########
@@ -759,18 +759,19 @@ void addSubquery(
Integer getDatastoreWriteTimeoutMillis();
/**
- * Method to cancel any executing queries. If the underlying datastore
doesn't support
- * cancellation of queries this will throw JDOUnsupportedOptionException. If
the cancellation
- * fails (e.g in the underlying datastore) then this will throw a
JDOException.
+ * Method to cancel all executions of this query instance. If the underlying
datastore doesn't
+ * support cancellation of queries this will throw
JDOUnsupportedOptionException. If the
+ * cancellation fails (e.g in the underlying datastore) then this will throw
a JDOException.
*
* @since 3.0
*/
void cancelAll();
/**
- * Method to cancel an executing query in the specified thread. If the
underlying datastore
- * doesn't support cancellation of queries this will throw
JDOUnsupportedOptionException. If the
- * cancellation fails (e.g in the underlying datastore) then this will throw
a JDOException.
+ * Method to cancel any executions of this query instance that run in the
specified thread. If the
+ * underlying datastore doesn't support cancellation of queries this will
throw
+ * JDOUnsupportedOptionException. If the cancellation fails (e.g in the
underlying datastore) then
+ * this will throw a JDOException.
*
* @param thread The thread to cancel
Review Comment:
Even so I think it would still be more clear to say “of this query instance”
instead of “an executing query”
--
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
