Gabriella Lotz has posted comments on this change. ( http://gerrit.cloudera.org:8080/23655 )
Change subject: Add Introducing REST API blog post ...................................................................... Patch Set 2: (12 comments) http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md File _posts/2025-11-10-introducing-the-rest-api.md: http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@3 PS1, Line 3: REST API for metadata m > Not sure what was the final naming that we agreed on, but I think it would Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@7 PS1, Line 7: or metad > REST API for metadata management Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@7 PS1, Line 7: client applicati > not sure if this is needed or just saying client applications is enough. Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@13 PS1, Line 13: The Apache Kudu REST API exposes table-metadata and administrative operations over HTTP on top of Kudu's catalog. It lets users handle these tasks with lightweight REST calls instead of building a full client application. > I think its not necessary to mention explicitly the json response part. I w Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@21 PS1, Line 21: : ## API Overview > I would maybe say that remove these two bullet points from here. (In my opi Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@26 PS1, Line 26: > maybe mention that this is available on masters webserver on this path. Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@37 PS1, Line 37: **GET** `/api/v1/tabl > IIUC these are now two responses. Without the request path/method it lacks Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@97 PS1, Line 97: : ### Metadata Only > While thinking about this I've figured that it would be nice to have the Sw Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@100 PS1, Line 100: > As mentioned above, I think a "Limitations" section would maybe be better h Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@102 PS1, Line 102: : The REST API must be explicitly enabled using the `--enable_rest_api` flag on Kudu masters. The master's webserver mus > In case you go with the "Limitations" approach this could be removed from t Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@106 PS1, Line 106: > Maybe mention here that users can get the leader address by using the leade Done http://gerrit.cloudera.org:8080/#/c/23655/1/_posts/2025-11-10-introducing-the-rest-api.md@111 PS1, Line 111: > Does it make sense to mention the webserver_enabled flag as well? Done -- To view, visit http://gerrit.cloudera.org:8080/23655 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: gh-pages Gerrit-MessageType: comment Gerrit-Change-Id: I1435431857a9f493e6492fae002c70f7af4cdd6c Gerrit-Change-Number: 23655 Gerrit-PatchSet: 2 Gerrit-Owner: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Gabriella Lotz <[email protected]> Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Comment-Date: Fri, 14 Nov 2025 14:15:16 +0000 Gerrit-HasComments: Yes
