add javadoc for storage queue and messaging #27532
Conversation
|
API changes have been detected in |
|
API changes have been detected in |
|
API changes have been detected in |
| @@ -62,7 +62,7 @@ protected void doStart() { | |||
|
|
|||
| @Override | |||
| protected void doStop() { | |||
| this.delegate.start(); | |||
| @@ -52,6 +52,10 @@ public String getConnectionString() { | |||
| return connectionString; | |||
| } | |||
|
|
|||
| /** | |||
| * Set the connection string to connect to the service. | |||
There was a problem hiding this comment.
better add an empty line here
There was a problem hiding this comment.
Hey, I noticed that most of the setters' javadoc in our xxProperties.java don't have that empty line, need we follow that?
There was a problem hiding this comment.
We have a lot of things to do to improve the java doc.
6 month ago, I create an issue: #24332, which is intended to do this. But this issue is closed without satisfy the requirement.
Since we are in hurry to release 4.0, we do not have enough time to improve the java doc. My current suggestions is improve the java doc's quality for new classes and new public methods.
We can get a lot of documents by searching by google. For example: https://www.oracle.com/technical-resources/articles/java/javadoc-tool.html
As subject, to resolve API review issues.