ignite-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [ignite] pavlukhin commented on a change in pull request #6760: IGNITE-12051: update javadoc for IgniteKernal class
Date Sun, 11 Aug 2019 05:54:56 GMT
pavlukhin commented on a change in pull request #6760: IGNITE-12051: update javadoc for IgniteKernal
class
URL: https://github.com/apache/ignite/pull/6760#discussion_r312723872
 
 

 ##########
 File path: modules/core/src/main/java/org/apache/ignite/internal/IgniteKernal.java
 ##########
 @@ -293,52 +294,79 @@
 import static org.apache.ignite.lifecycle.LifecycleEventType.BEFORE_NODE_START;
 
 /**
- * Ignite kernal.
- * <p/>
- * See <a href="http://en.wikipedia.org/wiki/Kernal">http://en.wikipedia.org/wiki/Kernal</a>
for information on the
- * misspelling.
+ * This class represents an implementation of the extended main Ignite API (called {@link
IgniteEx}). It also controls
 
 Review comment:
   I think it worth to have a link to `Ignite` interface here as the current class is the
real implementation of it. `IgniteEx` could be mentioned as well with a note that it adds
several methods to `Ignite` for internal purposes. What do you think?

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


With regards,
Apache Git Services

Mime
View raw message