Copilot commented on code in PR #7365:
URL: https://github.com/apache/kyuubi/pull/7365#discussion_r2972813860
##########
docs/client/bi_tools/hue.md:
##########
@@ -25,7 +25,7 @@
### Get Kyuubi Started
-[Get the server Started](../../quick_start/quick_start.html) first before your
try Hue with Kyuubi.
+[Get the server Started](../../quick_start/quick_start.rst) first before your
try Hue with Kyuubi.
Review Comment:
The sentence has a grammatical typo: “before your try Hue…” should be
“before you try Hue…”.
```suggestion
[Get the server Started](../../quick_start/quick_start.rst) first before you
try Hue with Kyuubi.
```
##########
docs/deployment/engine_lifecycle.md:
##########
@@ -51,7 +51,7 @@ To better improve the overall resource utilization of the
cluster,
The above two configurations can be used together to set the TTL of engines.
These configurations are user-facing and able to use in JDBC urls.
-Note that, for [connection](engine_share_level.html#connection) share level
engines that will be terminated at once when the connection is disconnected,
these configurations not necessarily work in this case.
+Note that, for [connection](engine_share_level.md#connection) share level
engines that will be terminated at once when the connection is disconnected,
these configurations not necessarily work in this case.
Review Comment:
Grammar: “these configurations not necessarily work” should be “these
configurations do not necessarily work”.
```suggestion
Note that, for [connection](engine_share_level.md#connection) share level
engines that will be terminated at once when the connection is disconnected,
these configurations do not necessarily work in this case.
```
##########
docs/client/rest/rest_api.md:
##########
@@ -455,7 +455,7 @@ Refresh the Hadoop configurations of the Kyuubi server.
### POST /admin/refresh/user_defaults_conf
-Refresh the [user defaults
configs](../../configuration/settings.html#user-defaults) with key in format in
the form of `___{username}___.{config key}` from default property file.
+Refresh the [user defaults
configs](../../configuration/settings.md#user-defaults) with key in format in
the form of `___{username}___.{config key}` from default property file.
Review Comment:
Wording is currently redundant/unclear: “with key in format in the form of
…” repeats itself. Consider rephrasing to a single clear clause describing the
expected key format.
```suggestion
Refresh the [user defaults
configs](../../configuration/settings.md#user-defaults) whose keys have the
form `___{username}___.{config key}` from the default property file.
```
##########
docs/client/bi_tools/datagrip.md:
##########
@@ -29,7 +29,7 @@ Please go to [Download
DataGrip](https://www.jetbrains.com/datagrip/download) to
### Get Kyuubi Started
-[Get kyuubi server started](../../quick_start/quick_start.html) before you try
DataGrip with kyuubi.
+[Get kyuubi server started](../../quick_start/quick_start.rst) before you try
DataGrip with kyuubi.
Review Comment:
Brand/capitalization typo in the link sentence: “Get kyuubi server started”
should use “Kyuubi” (and ideally consistent capitalization).
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
