hudi icon indicating copy to clipboard operation
hudi copied to clipboard

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

Open sagarlakshmipathy opened this issue 1 year ago • 1 comments

Refer: https://github.com/apache/hudi/pull/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

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

sagarlakshmipathy avatar Dec 11 '23 03:12 sagarlakshmipathy

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 :)

bhasudha avatar Dec 12 '23 13:12 bhasudha

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

yihua avatar Mar 09 '24 16:03 yihua

could resolve merge conflict - so raised this https://github.com/apache/hudi/pull/10940

sagarlakshmipathy avatar Mar 30 '24 04:03 sagarlakshmipathy