Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

rename and update COPY page #19310

Open
wants to merge 7 commits into
base: main
Choose a base branch
from
Open

rename and update COPY page #19310

wants to merge 7 commits into from

Conversation

taroface
Copy link
Contributor

@taroface taroface commented Jan 16, 2025

DOC-12056
DOC-6972

  • Renamed COPY FROM page to COPY (v23.1-v25.1)
  • Documented COPY TO syntax and added an example

This will depend on a cockroach PR to update the diagram: cockroachdb/cockroach#139365.

This PR affects many pages, due to the link changes. The doc to review is copy.html, i.e. Preview

Copy link

github-actions bot commented Jan 16, 2025

Files changed:

Copy link

netlify bot commented Jan 16, 2025

Deploy Preview for cockroachdb-interactivetutorials-docs canceled.

Name Link
🔨 Latest commit 01a5843
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-interactivetutorials-docs/deploys/678fd8e0fc4a1e00085c2c32

Copy link

netlify bot commented Jan 16, 2025

Deploy Preview for cockroachdb-api-docs canceled.

Name Link
🔨 Latest commit 01a5843
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-api-docs/deploys/678fd8e0dcdf780007e8662e

Copy link

netlify bot commented Jan 16, 2025

Netlify Preview

Name Link
🔨 Latest commit 01a5843
🔍 Latest deploy log https://app.netlify.com/sites/cockroachdb-docs/deploys/678fd8e0a5f7c60008fb63e7
😎 Deploy Preview https://deploy-preview-19310--cockroachdb-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@taroface taroface requested a review from mgartner January 21, 2025 19:47
craig bot pushed a commit to cockroachdb/cockroach that referenced this pull request Jan 31, 2025
139365: docgen: update COPY TO STDOUT portions of SQL diagram r=mgartner a=taroface

Updated `COPY` diagram so that it better reflects the `COPY TO` syntax that was added in 23.1 (this PR will be backported to every version up until 23.1). These changes accompany the docs updates in cockroachdb/docs#19310
 
Current diagram:

<img width="647" alt="image" src="https://github.com/user-attachments/assets/eb70b831-3473-40aa-823f-5432a8cf2698" />

New diagram:

<img width="829" alt="image" src="https://github.com/user-attachments/assets/314e1345-cec8-4a72-9be2-fb4b2f2885ee" />

Epic: none
Release note: none
Release justification: non-production code change

Co-authored-by: Ryan Kuo <[email protected]>
Copy link
Contributor

@mgartner mgartner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating this! LGTM

craig bot pushed a commit to cockroachdb/cockroach that referenced this pull request Jan 31, 2025
139256: sql/rowenc: reduce index key prefix calls r=annrpom a=annrpom

### sql/rowenc: reduce index key prefix calls
This patch removes redundant calls to `MakeIndexKeyPrefix` during
the construction of `IndexEntry`s by saving each first-time call in a
map that we can later lookup. Previously, we would make this call
for each row; however, as the prefix (table id + index id) for a
particular index remains the same, we do not need to do any
recomputation.

Epic: [CRDB-42901](https://cockroachlabs.atlassian.net/browse/CRDB-42901)
Fixes: #137798

Release note: None

---

### sql/rowexec: run BenchmarkIndexBackfill on-disk
Epic: none
Release note: None


139365: docgen: update COPY TO STDOUT portions of SQL diagram r=mgartner a=taroface

Updated `COPY` diagram so that it better reflects the `COPY TO` syntax that was added in 23.1 (this PR will be backported to every version up until 23.1). These changes accompany the docs updates in cockroachdb/docs#19310
 
Current diagram:

<img width="647" alt="image" src="https://github.com/user-attachments/assets/eb70b831-3473-40aa-823f-5432a8cf2698" />

New diagram:

<img width="829" alt="image" src="https://github.com/user-attachments/assets/314e1345-cec8-4a72-9be2-fb4b2f2885ee" />

Epic: none
Release note: none
Release justification: non-production code change

140071: storage: covert SizeSpec to protobuf r=RaduBerinde a=andrewbaptist

This commit moves SizeSpec to the storagepb directory and converts it toa protobuf. Additionally it pulls shared storage into the global StorageConfig file.

Epic: none

Release note: None

Co-authored-by: Annie Pompa <[email protected]>
Co-authored-by: Ryan Kuo <[email protected]>
Co-authored-by: Andrew Baptist <[email protected]>
craig bot pushed a commit to cockroachdb/cockroach that referenced this pull request Jan 31, 2025
139365: docgen: update COPY TO STDOUT portions of SQL diagram r=mgartner a=taroface

Updated `COPY` diagram so that it better reflects the `COPY TO` syntax that was added in 23.1 (this PR will be backported to every version up until 23.1). These changes accompany the docs updates in cockroachdb/docs#19310
 
Current diagram:

<img width="647" alt="image" src="https://github.com/user-attachments/assets/eb70b831-3473-40aa-823f-5432a8cf2698" />

New diagram:

<img width="829" alt="image" src="https://github.com/user-attachments/assets/314e1345-cec8-4a72-9be2-fb4b2f2885ee" />

Epic: none
Release note: none
Release justification: non-production code change

140071: storage: covert SizeSpec to protobuf r=RaduBerinde a=andrewbaptist

This commit moves SizeSpec to the storagepb directory and converts it toa protobuf. Additionally it pulls shared storage into the global StorageConfig file.

Epic: none

Release note: None

140241: multiregionccl: deflake TestColdStartLatency r=rafiss a=rafiss

Similar to what we're seeing in
#140172 and the issues linked to #140187, this test is flaky due to running with autocommit_before_ddl=true in a multitenant setup.

fixes #140174
Release note: None

Co-authored-by: Ryan Kuo <[email protected]>
Co-authored-by: Andrew Baptist <[email protected]>
Co-authored-by: Rafi Shamim <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants