Skip to content

[MINOR] [DOCS] changes to redshift & starrocks compat matrix#10294

Closed
sagarlakshmipathy wants to merge 9 commits intoapache:asf-sitefrom
sagarlakshmipathy:update-compatibility-matrix
Closed

[MINOR] [DOCS] changes to redshift & starrocks compat matrix#10294
sagarlakshmipathy wants to merge 9 commits intoapache:asf-sitefrom
sagarlakshmipathy:update-compatibility-matrix

Conversation

@sagarlakshmipathy
Copy link
Contributor

Refer: #10202

Change Logs

Update SQL Queries compatibility matrix
Redshift spectrum support Hudi MOR RO tables but doesn't support Snapshot and Incremental queries.
StarRocks support MOR RO and Snapshot queries, but doesn't support Incremental queries.
Made some cosmetic changes that intellij suggested.

Impact

Doc update.

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

MINOR Doc update.

Documentation Update

Doc update.

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable - npm run build and npm run serve ran successfully
  • [NA] CI passed - CI runs after raising PR

@sagarlakshmipathy sagarlakshmipathy changed the title changes to redshift & starrocks compat matrix [MINOR] [DOCS] changes to redshift & starrocks compat matrix Dec 11, 2023
@bhasudha
Copy link
Contributor

minor nit: Please avoid intellij suggested or whitespace changes going forward. Since this can be different across individual person's settings. And gets in the way of review :)

@github-actions github-actions bot added docs size:XL PR with lines of changes > 1000 labels Feb 26, 2024
@yihua
Copy link
Contributor

yihua commented Mar 9, 2024

@sagarlakshmipathy could you address the comments when you have time?

@sagarlakshmipathy
Copy link
Contributor Author

could resolve merge conflict - so raised this #10940

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XL PR with lines of changes > 1000

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants