SNOW-1518540 release public preview

[sfc-gh-zli]

Overview

SNOW-1518540 release public preview

Pre-review checklist

• I attest that this change meets the bar for low risk without security requirements as defined in the Accelerated Risk Assessment Criteria and I have taken the Risk Assessment Training in Workday.
  • Checking this checkbox is mandatory if using the Accelerated Risk Assessment to risk assess the changes in this Pull Request.
  • If this change does not meet the bar for low risk without security requirements (as confirmed by the peer reviewers of this pull request) then a formal Risk Assessment must be completed. Please note that a formal Risk Assessment will require you to spend extra time performing a security review for this change. Please account for this extra time earlier rather than later to avoid unnecessary delays in the release process.
• This change has passed precommit
• This change is TEST-ONLY
• This change has been through a design review process and the test plan is signed off by the owner
• This change is parameter protected
• This change has code coverage for the new code added

[sfc-gh-vpai]

I think the top-level README.md probably needs to change as well since it refers to the private-preview directory

[sfc-gh-igarish]

Are we keeping X- header here?

[sfc-gh-zli]

yes, this field is not internal. It's meant to be public facing.

[sfc-gh-igarish]

LGTM

[sfc-gh-wshangguan]

For some specs, e.g. dynamic table, is it the manual created version or the one generated by DSL?

[sfc-gh-zli]

For some specs, e.g. dynamic table, is it the manual created version or the one generated by DSL?

It's the one in 8.30 release branch. I think it is generated, because it has x-enum-varname.