You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2022/08/09 08:17:50 UTC

[GitHub] [pulsar] tisonkun opened a new pull request, #17010: [improve][doc] Separate language-specific and language-agnostic clients

tisonkun opened a new pull request, #17010:
URL: https://github.com/apache/pulsar/pull/17010

   Related to https://github.com/apache/pulsar-site/issues/146.
   
   ### Motivation
   
   * Websocket is not a language but a language-agnostic interface.
   
   ### Modifications
   
   1. Separate language-specific and language-agnostic clients.
   2. Improve third-party client libraries display - show stars, grouped by languages.
   3. Avoid redundant links to doc tools, it's not what users care.
   
   ### Documentation
     
   - [x] `doc` 
   
   cc @Anonymitaet @momo-jun @shibd @RobertIndie 


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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941300537


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   Fixed at https://github.com/apache/pulsar/pull/17010/commits/e5c403f8e7f52b84446cc0fef392dfbe58d724b3.



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941095703


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   @blankensteiner do you mean that we list pulsar-dotpulsar twice, once under the C# category, and the other under F# category?



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


[GitHub] [pulsar] Lanayx commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
Lanayx commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941219992


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   As for pulsar-client-dotnet there is no need to separate C# and F#, since user experience is the same when using the library from both languages. .NET is preferable choice for me, also there is a Node.js section below which is not a language either. If user is interested in the project language he can simply click the link and view the repo.



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


[GitHub] [pulsar] Lanayx commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
Lanayx commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941227596


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   Both Kafka and RabbitMQ refer to clients as .NET clients
   https://www.rabbitmq.com/devtools.html
   https://cwiki.apache.org/confluence/display/KAFKA/Clients



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941132139


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   @blankensteiner I get your point. Will keep the content as is.
   
   I think that's why [Redis client page](https://redis.io/docs/clients/) list `.NET` language family as `C#`.
   
   If @Lanayx has a different opinion to list `pulsar-client-dotnet` under F# as it's written in F#, I'll make the change then.



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941974828


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-

Review Comment:
   In another word, users don't care about how we organize release note and codebase, if only the link directs to the right place.



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941078444


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   I was thinking about whether `.NET` is a language. [Redis client page](https://redis.io/docs/clients/) list `.NET` language family as `C#`. I'm unsure what's the conventions of C# community.
   
   cc @blankensteiner @Lanayx



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


[GitHub] [pulsar] RobertIndie merged pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
RobertIndie merged PR #17010:
URL: https://github.com/apache/pulsar/pull/17010


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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941973584


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |

Review Comment:
   OK. Updated at 74ea3719a89e6a25bf82de7b3922be8fac7965b1.



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941043282


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |

Review Comment:
   API docs can be found in user doc, we don't have to inline it with `<br/>` here.



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


[GitHub] [pulsar] Lanayx commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
Lanayx commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941227596


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   Both Kafka and RabbitMQ refer to client as .NET clients
   https://www.rabbitmq.com/devtools.html
   https://cwiki.apache.org/confluence/display/KAFKA/Clients



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


[GitHub] [pulsar] RobertIndie commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
RobertIndie commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941063984


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   How about calling it `.Net`?   And for the official client as well. 
   `pulsar-client-dotnet` is written in F# and the official one `pulsar-dotpulsar` is written in C#. But they are compatible with all .Net languages such as C#/F#/VB.



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941974828


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-

Review Comment:
   In another word, users don't care about how we organize release note and codebase, if only the link directs to the right place/result.



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


[GitHub] [pulsar] blankensteiner commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
blankensteiner commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941125997


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   @tisonkun Many languages support calling assemblies written in other languages. So, if you want to list all languages and list which clients can be used from the languages, then you would need the C# section to also include the C/C++ client and F# client since that is technically possible. Therefore it is easier to use the language to tell what language the client is written in. Then people using F# can browse for an F# client, but also know that they can use a C# client if they want (and the other way around).



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


[GitHub] [pulsar] blankensteiner commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
blankensteiner commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941084726


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   Hi @tisonkun - it would make sense to list C# (DotPulsar) and F# as separate languages even though they share the .NET runtime.



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


[GitHub] [pulsar] Lanayx commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
Lanayx commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941227596


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   Both Kafka and RabbitMQ refer to client as .NET clients
   https://www.rabbitmq.com/devtools.html
   https://docs.confluent.io/platform/current/clients/index.html



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941974433


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-

Review Comment:
   > Also keep the whole explanation (lines 21 -22)
   
   Concise is important to document. We don't have to write down all information.
   
   Users don't think of what the word means at first but click the link, and the "standalone" result doesn't confuse. While the "bundled" result is explained by its result - it's bundled with the main repo / main release.



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941277142


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-
-:::
+| Interface | Documentation                             | Release note               | Code repo                                                                |
+| --------- | ----------------------------------------- | -------------------------- | ------------------------------------------------------------------------ |
+| REST      | [User doc](client-libraries-rest.md)      | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-broker)    |
+| WebSocket | [User doc](client-libraries-websocket.md) | [Bundled](/release-notes/) | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-websocket) |
 
 ## Feature matrix
+
 Pulsar client feature matrix for different languages is listed on [Pulsar Feature Matrix (Client and Function)](https://docs.google.com/spreadsheets/d/1YHYTkIXR8-Ql103u-IMI18TXLlGStK8uJjDsOOA0T20/edit#gid=1784579914) page.
 
 ## Third-party clients
 
 Besides the official released clients, multiple projects on developing Pulsar clients are available in different languages.
 
-> If you have developed a new Pulsar client, feel free to submit a pull request and add your client to the list below.
-
-| Language | Project | Maintainer | License | Description |
-|----------|---------|------------|---------|-------------|
-| Go | [pulsar-client-go](https://github.com/Comcast/pulsar-client-go) | [Comcast](https://github.com/Comcast) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | A native golang client |
-| Go | [go-pulsar](https://github.com/t2y/go-pulsar) | [t2y](https://github.com/t2y) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | 
-| Haskell | [supernova](https://github.com/cr-org/supernova) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Native Pulsar client for Haskell | 
-| Scala | [neutron](https://github.com/cr-org/neutron) | [Chatroulette](https://github.com/cr-org) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Purely functional Apache Pulsar client for Scala built on top of Fs2 |
-| Scala | [pulsar4s](https://github.com/sksamuel/pulsar4s) | [sksamuel](https://github.com/sksamuel) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Idomatic, typesafe, and reactive Scala client for Apache Pulsar |
-| Rust | [pulsar-rs](https://github.com/wyyerd/pulsar-rs) | [Wyyerd Group](https://github.com/wyyerd) | [![License](https://img.shields.io/badge/License-Apache%202.0-blue.svg)](https://opensource.org/licenses/Apache-2.0) | Future-based Rust bindings for Apache Pulsar |
-| .NET | [pulsar-client-dotnet](https://github.com/fsharplang-ru/pulsar-client-dotnet) | [Lanayx](https://github.com/Lanayx) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native .NET client for C#/F#/VB |
-| Node.js | [pulsar-flex](https://github.com/ayeo-flex-org/pulsar-flex) | [Daniel Sinai](https://github.com/danielsinai), [Ron Farkash](https://github.com/ronfarkash), [Gal Rosenberg](https://github.com/galrose)| [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native Nodejs client |
-| PHP | [pulsar-client-php](https://github.com/ikilobyte/pulsar-client-php) | [ikilobyte](https://github.com/ikilobyte) | [![GitHub](https://img.shields.io/badge/license-MIT-green.svg)](https://opensource.org/licenses/MIT) | Native PHP client |
+> Want your repository listed here? Just submit a PR to the [pulsar repository](https://github.com/apache/pulsar/edit/master/site2/docs/client-libraries.md).
+
+### C#

Review Comment:
   @Lanayx thanks for your feedback. Will push a commit later.



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941974433


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-

Review Comment:
   > Also keep the whole explanation (lines 21 -22)
   
   Concise is important to document. We don't have to write down all information.
   
   Users don't think of what the word means at first but click the link, and the "standalone" result doesn't confuse. While the "bundled" result is explained by its result - it's along with the main repo / main release.



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


[GitHub] [pulsar] Anonymitaet commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941962182


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |

Review Comment:
   Suggest keeping API doc links in the table, because it makes the info more prominently since they are frequently used by developers. 



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


[GitHub] [pulsar] Anonymitaet commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941960865


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-

Review Comment:
   Suggest keeping lines 21-22. Reasons:
   
   1. It's more clear to show users the whole information (lines 21-22), especially for those who are not familiar with Pulsar.
   
   2. Some users can not fully understand the meaning of `standalone` and `bundled` because they can represent other meanings as well, or at least, they can not understand it at first glance and need to take a little more time to do the guesswork.
   
   🔹🔹🔹
   
   So how about compromising it as:
   
   - Keep `standalone` and `bundled` in tables
   - Also keep the whole explanation (lines 21 -22)
   
   In this way, there is no redundant, missing, or confusing info.
   



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


[GitHub] [pulsar] tisonkun commented on a diff in pull request #17010: [improve][doc] Separate language-specific and language-agnostic clients

Posted by GitBox <gi...@apache.org>.
tisonkun commented on code in PR #17010:
URL: https://github.com/apache/pulsar/pull/17010#discussion_r941044005


##########
site2/docs/client-libraries.md:
##########
@@ -4,42 +4,74 @@ title: Pulsar client libraries
 sidebar_label: "Overview"
 ---
 
-Pulsar supports the following client libraries:
+Pulsar supports the following language specific client libraries:
 
-| Language  | Documentation                                                                 | Release note                                                             | Code repo                                                                     |
-|-----------|-------------------------------------------------------------------------------|--------------------------------------------------------------------------|-------------------------------------------------------------------------------|
-| Java      | - [User doc](client-libraries-java.md) <br/><br/> - [API doc](/api/client/)   | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
-| C++       | - [User doc](client-libraries-cpp.md) <br/><br/> - [API doc](/api/cpp/)       | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
-| Python    | - [User doc](client-libraries-python.md) <br/><br/> - [API doc](/api/python/) | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
-| WebSocket | [User doc](client-libraries-websocket.md)                                     | [Here](/release-notes/)                                                  | [Here](https://github.com/apache/pulsar/tree/master/pulsar-websocket)         |
-| Go client | [User doc](client-libraries-go.md)                                            | [Here](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-client-go)                            |
-| Node.js   | [User doc](client-libraries-node.md)                                          | [Here](https://github.com/apache/pulsar-client-node/releases)            | [Here](https://github.com/apache/pulsar-client-node)                          |
-| C#        | [User doc](client-libraries-dotnet.md)                                        | [Here](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Here](https://github.com/apache/pulsar-dotpulsar)                            |
+| Language  | Documentation                          | Release note                                                                   | Code repo                                                                        |
+| --------- | -------------------------------------- | ------------------------------------------------------------------------------ | -------------------------------------------------------------------------------- |
+| Java      | [User doc](client-libraries-java.md)   | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client)            |
+| C++       | [User doc](client-libraries-cpp.md)    | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp)        |
+| Python    | [User doc](client-libraries-python.md) | [Bundled](/release-notes/)                                                     | [Bundled](https://github.com/apache/pulsar/tree/master/pulsar-client-cpp/python) |
+| Go client | [User doc](client-libraries-go.md)     | [Standalone](https://github.com/apache/pulsar-client-go/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-client-go)                         |
+| Node.js   | [User doc](client-libraries-node.md)   | [Standalone](https://github.com/apache/pulsar-client-node/releases)            | [Standalone](https://github.com/apache/pulsar-client-node)                       |
+| C#        | [User doc](client-libraries-dotnet.md) | [Standalone](https://github.com/apache/pulsar-dotpulsar/blob/master/CHANGELOG) | [Standalone](https://github.com/apache/pulsar-dotpulsar)                         |
 
-:::note
+Pulsar supports the following language-agnostic client libraries:
 
-- The code repos of **Java, C++, Python,** and **WebSocket** clients are hosted in the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are released with Pulsar, so their release notes are parts of [Pulsar release note](/release-notes/).
-- The code repos of **Go, Node.js,** and **C#** clients are hosted outside of the [Pulsar main repo](https://github.com/apache/pulsar) and these clients are not released with Pulsar, so they have independent release notes.
-

Review Comment:
   This note is included as "bundled" or "standalone".



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