Skip to content

Comments

[HUDI-5901][DOCS] Added links to understand in what cases HUDI Big Query integration works#8091

Merged
danny0405 merged 2 commits intoapache:asf-sitefrom
vinaykv1991:patch-1
Mar 8, 2023
Merged

[HUDI-5901][DOCS] Added links to understand in what cases HUDI Big Query integration works#8091
danny0405 merged 2 commits intoapache:asf-sitefrom
vinaykv1991:patch-1

Conversation

@vinaykv1991
Copy link

@vinaykv1991 vinaykv1991 commented Mar 3, 2023

Why changes - In what cases the integration works is clear, finding elaboration of that cases can be made easier with links. What are the Changes - Added links to understand in what cases HUDI Big Query integration works.

This change will help user to understand the restriction of integration faster.

Change Logs

Why changes - In what cases the integration works is clear, finding elaboration of that cases can be made easier with links. What are the Changes - Added links to understand in what cases HUDI Big Query integration works.

This change will help user to understand the restriction of integration faster.

Impact

No impact to performace or user. Documentation change.

Risk level (write none, low medium or high below)

None

Documentation Update

This changes the https://hudi.apache.org/docs/gcp_bigquery/

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

Why changes - In what cases the integration works is clear, finding elaboration of that cases can be made easier with links.
What are the Changes - Added links to understand in what cases HUDI Big Query integration works.

This change will help user to understand the restriction of integration faster.
@vinaykv1991 vinaykv1991 changed the title Update gcp_bigquery.md [DOCS] Update gcp_bigquery.md Mar 3, 2023
@vinaykv1991
Copy link
Author

@bhasudha @nfarah86 @codope please review.

@danny0405 danny0405 changed the title [DOCS] Update gcp_bigquery.md [DOCS] Added links to understand in what cases HUDI Big Query integration works Mar 4, 2023
@danny0405 danny0405 added the docs label Mar 4, 2023
Fixing build failure 

Error:  Error: Docusaurus found broken links!

Please check the pages of your site in the list below, and make sure you don't reference any path that does not exist.
Note: it's possible to ignore broken links with the 'onBrokenLinks' Docusaurus configuration, and let the build pass.

Exhaustive list of all broken links found:

- On source page path = /docs/querying_data:
   -> linking to /docs/next/query_engine_setup#redshift-spectrum (resolved as: /docs/next/query_engine_setup)
   -> linking to /docs/next/query_engine_setup#doris (resolved as: /docs/next/query_engine_setup)
   -> linking to /docs/next/query_engine_setup#starrocks (resolved as: /docs/next/query_engine_setup)

    at reportMessage (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/utils/lib/index.js:306:19)
    at handleBrokenLinks (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/core/lib/server/brokenLinks.js:138:35)
    at async buildLocale (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/core/lib/commands/build.js:155:5)
    at async tryToBuildLocale (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/core/lib/commands/build.js:33:20)
    at async mapAsyncSequencial (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/utils/lib/index.js:262:24)
    at async build (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/core/lib/commands/build.js:68:25)
Error: Process completed with exit code 1.
@vinaykv1991
Copy link
Author

@danny0405 fixed a broken links in querying_data.md for build failure. Please review

@danny0405 danny0405 changed the title [DOCS] Added links to understand in what cases HUDI Big Query integration works [HUDI-5901][DOCS] Added links to understand in what cases HUDI Big Query integration works Mar 8, 2023
@danny0405 danny0405 merged commit 4911227 into apache:asf-site Mar 8, 2023
Zouxxyy pushed a commit to Zouxxyy/hudi that referenced this pull request Apr 3, 2023
…ery integration works (apache#8091)

* Update gcp_bigquery.md

Why changes - In what cases the integration works is clear, finding elaboration of that cases can be made easier with links.
What are the Changes - Added links to understand in what cases HUDI Big Query integration works.

This change will help user to understand the restriction of integration faster.

* Update querying_data.md

Fixing build failure 

Error:  Error: Docusaurus found broken links!

Please check the pages of your site in the list below, and make sure you don't reference any path that does not exist.
Note: it's possible to ignore broken links with the 'onBrokenLinks' Docusaurus configuration, and let the build pass.

Exhaustive list of all broken links found:

- On source page path = /docs/querying_data:
   -> linking to /docs/next/query_engine_setup#redshift-spectrum (resolved as: /docs/next/query_engine_setup)
   -> linking to /docs/next/query_engine_setup#doris (resolved as: /docs/next/query_engine_setup)
   -> linking to /docs/next/query_engine_setup#starrocks (resolved as: /docs/next/query_engine_setup)

    at reportMessage (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/utils/lib/index.js:306:19)
    at handleBrokenLinks (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/core/lib/server/brokenLinks.js:138:35)
    at async buildLocale (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/core/lib/commands/build.js:155:5)
    at async tryToBuildLocale (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/core/lib/commands/build.js:33:20)
    at async mapAsyncSequencial (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/utils/lib/index.js:262:24)
    at async build (/home/runner/work/hudi/hudi/website/node_modules/@docusaurus/core/lib/commands/build.js:68:25)
Error: Process completed with exit code 1.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants