[
https://issues.apache.org/jira/browse/DRILL-3661?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14727651#comment-14727651
]
ASF GitHub Bot commented on DRILL-3661:
---------------------------------------
Github user dsbos commented on a diff in the pull request:
https://github.com/apache/drill/pull/119#discussion_r38558416
--- Diff:
exec/jdbc/src/main/java/org/apache/drill/jdbc/impl/DrillCursor.java ---
@@ -66,17 +69,18 @@
private boolean afterFirstBatch = false;
/**
- * Whether the next call to this.next() should just return {@code true}
rather
- * than calling nextRowInternally() to try to advance to the next
- * record.
+ * Whether the next call to {@code this.}{@link #next()} should just
return
+ * {@code true} rather than calling {@link #nextRowInternally()} to try
to
+ * advance to the next record.
* <p>
- * Currently, can be true only for first call to next().
+ * Currently, can be true only for first call to {@link #next()}.
* </p>
* <p>
- * (Relates to loadInitialSchema()'s calling nextRowInternally()
+ * (Relates to {@link #loadInitialSchema()}'s calling
nextRowInternally()
--- End diff --
Frequently, for second and later references in the same paragraph or
adjacent paragraphs, I avoid making those later references links, since having
more links can make the text less readable.
However, since here I already have multiple links to next(), I did make
that nextRowInternally() a link (and added a missing {@code...} for
"Statement.execute...(...)").
> Add/edit various JDBC Javadoc.
> ------------------------------
>
> Key: DRILL-3661
> URL: https://issues.apache.org/jira/browse/DRILL-3661
> Project: Apache Drill
> Issue Type: Bug
> Reporter: Daniel Barclay (Drill)
> Assignee: Daniel Barclay (Drill)
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
