Copilot commented on code in PR #117:
URL: https://github.com/apache/gravitino-site/pull/117#discussion_r2910057489
##########
docs/sidebars.ts:
##########
@@ -103,6 +118,11 @@ const sidebars: SidebarsConfig = {
label: 'Web UI',
type: 'doc'
},
+ {
+ id: 'webui-v2',
+ label: 'Web UI v2',
+ type: 'doc'
+ },
{
Review Comment:
The sidebar references a new doc id `webui-v2`, but there is no matching doc
file in the repo (e.g., `docs/webui-v2.md(x)` or a versioned doc). This will
create a broken link; please add the doc or remove/rename this sidebar entry to
match an existing doc id.
```suggestion
```
##########
docs/sidebars.ts:
##########
@@ -243,6 +273,47 @@ const sidebars: SidebarsConfig = {
}
]
},
+ {
+ label: 'Table Maintenance Service',
+ type: 'category',
+ items: [
+ {
+ id: 'table-maintenance-service/optimizer',
+ label: 'Optimizer',
+ type: 'doc'
+ },
+ {
+ id: 'table-maintenance-service/optimizer-cli-reference',
+ label: 'Optimizer CLI reference',
+ type: 'doc'
+ },
+ {
+ id: 'table-maintenance-service/optimizer-configuration',
+ label: 'Optimizer configuration',
+ type: 'doc'
+ },
+ {
+ id: 'table-maintenance-service/optimizer-extension-guide',
+ label: 'Optimizer extension guide',
+ type: 'doc'
+ },
+ {
+ id: 'table-maintenance-service/optimizer-quick-start',
+ label: 'Optimizer quick start',
+ type: 'doc'
+ },
+ {
+ id: 'table-maintenance-service/optimizer-troubleshooting',
+ label: 'Optimizer troubleshooting',
+ type: 'doc'
+ },
+ {
+ id: 'iceberg-compaction-policy',
+ label: 'Iceberg compaction policy',
+ type: 'doc'
+ }
+ ]
Review Comment:
This new 'Table Maintenance Service' category references multiple doc ids
under `table-maintenance-service/*` plus `iceberg-compaction-policy`, but
neither the `docs/table-maintenance-service/` directory nor any of these doc
files exist in the repo. This will create broken links / doc build failures;
please add the missing docs or remove these sidebar entries until the docs land.
##########
docs/sidebars.ts:
##########
@@ -360,6 +431,11 @@ const sidebars: SidebarsConfig = {
label: 'Flink connector',
type: 'category',
items: [
+ {
+ id: 'flink-connector/flink-authentication-with-gravitino',
+ label: 'Flink authentication with Gravitino server',
+ type: 'doc'
+ },
{
Review Comment:
The sidebar adds `flink-connector/flink-authentication-with-gravitino`, but
there is no corresponding doc file in `docs/flink-connector/` (or elsewhere in
the repo). Please add the missing doc or remove/rename this sidebar entry to
match an existing document id.
```suggestion
```
##########
docs/sidebars.ts:
##########
@@ -540,6 +616,11 @@ const sidebars: SidebarsConfig = {
label: 'Manage model metadata',
type: 'doc'
},
+ {
+ id: 'manage-user-defined-function-using-gravitino',
+ label: 'Manage user defined function',
+ type: 'doc'
+ },
{
Review Comment:
The sidebar adds `manage-user-defined-function-using-gravitino`, but no
matching doc file exists in the repo (e.g.,
`docs/manage-user-defined-function-using-gravitino.md(x)`). This will create a
broken link; please add the doc or remove/rename this entry.
```suggestion
```
##########
docs/sidebars.ts:
##########
@@ -111,6 +131,11 @@ const sidebars: SidebarsConfig = {
label: 'Relational',
type: 'category',
items: [
+ {
+ id: 'jdbc-clickhouse-catalog',
+ label: 'ClickHouse catalog',
+ type: 'doc'
+ },
{
Review Comment:
The newly added sidebar entry `jdbc-clickhouse-catalog` does not have a
corresponding doc file anywhere in the repo (no
`docs/jdbc-clickhouse-catalog.md(x)` found). Docusaurus will render a broken
link; add the doc or remove/adjust the id to an existing document.
```suggestion
```
##########
docs/sidebars.ts:
##########
@@ -148,6 +173,11 @@ const sidebars: SidebarsConfig = {
id: 'lakehouse-generic-lance-table',
label: 'Lance table support',
type: 'doc'
+ },
+ {
+ id: 'lakehouse-generic-delta-table',
+ label: 'Delta table support',
+ type: 'doc'
Review Comment:
The sidebar adds `lakehouse-generic-delta-table`, but no corresponding doc
file exists in the repository. Please add
`docs/lakehouse-generic-delta-table.md(x)` (or the versioned equivalent) or
remove/rename this entry to match an existing doc id.
```suggestion
```
##########
docs/sidebars.ts:
##########
@@ -89,9 +89,24 @@ const sidebars: SidebarsConfig = {
type: 'doc'
},
{
- id: 'lance-rest-service',
label: 'Lance REST service',
- type: 'doc'
+ type: 'category',
+ link: {
+ type: 'doc',
+ id: 'lance-rest-service'
+ },
+ items: [
+ {
+ id: 'lance-rest-integration',
+ type: 'doc',
+ label: 'Lance integration'
+ },
+ {
+ id: 'lance-rest-server-chart',
+ type: 'doc',
+ label: 'Install Lance REST Server on Kubernetes'
+ }
+ ]
Review Comment:
The sidebar adds items for `lance-rest-integration` and
`lance-rest-server-chart`, but there are no corresponding doc files (no
`docs/lance-rest-integration.md(x)` or `docs/lance-rest-server-chart.md(x)`
found). This will produce broken links / doc build failures; either add the
docs or remove these items from the sidebar.
```suggestion
items: []
```
--
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]
