[GitHub] [pulsar] dave2wave commented on a change in pull request #14577: [Doc] add explanations for Java client library

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

dave2wave commented on a change in pull request #14577:
URL: https://github.com/apache/pulsar/pull/14577#discussion_r822861570



##########
File path: site2/docs/client-libraries-java.md
##########
@@ -12,7 +12,7 @@ Javadoc for the Pulsar client is divided into two domains by package as follows.
 
 Package | Description | Maven Artifact
 :-------|:------------|:--------------
-[`org.apache.pulsar.client.api`](/api/client) | The producer and consumer API | [org.apache.pulsar:pulsar-client:{{pulsar:version}}](http://search.maven.org/#artifactdetails%7Corg.apache.pulsar%7Cpulsar-client%7C{{pulsar:version}}%7Cjar)
+[`org.apache.pulsar.client.api`](/api/client) | [The producer and consumer API](https://pulsar.apache.org/api/client/) | [org.apache.pulsar:pulsar-client:{{pulsar:version}}](http://search.maven.org/#artifactdetails%7Corg.apache.pulsar%7Cpulsar-client%7C{{pulsar:version}}%7Cjar)

Review comment:
       It would be best for links from pages within the website domain to be `/api/client/` and leave out `https://pulsar.apache.org`. If you do that then you can easily test the links on staged websites!




-- 
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: commits-unsubscribe@pulsar.apache.org

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