Skip to content

Conversation

@Ian-Fogelman
Copy link
Contributor

@Ian-Fogelman Ian-Fogelman commented Oct 30, 2025

  • This fixes a typo mentioned in Issue found on page 'CREATE TABLE Statement' #6007 in the create table railroad diagram, it updates the type -> table. See before/after screenshots for clarification.
  • I know we are not updating preview now, but figured it would be a good fix for future cases.
  • Also just a note, was having a hard time to load these changes in the localhost browser, it took several restarts to see the changes in the railroad diagram. Potential caching issue/dynamic build issue?

Before:

image

After:

image

Links:

http://localhost:4000/docs/stable/sql/statements/create_table

@Ian-Fogelman Ian-Fogelman changed the title Update RR Files Update Create Table RR Files Oct 30, 2025
@szarnyasg
Copy link
Collaborator

Thanks!

Also just a note, was having a hard time to load these changes in the localhost browser, it took several restarts to see the changes in the railroad diagram. Potential caching issue/dynamic build issue?

Hi, yes, Jekyll's incremental builds are not great. I usually just do git add on whatever I'm working on, then git clean and a full rebuild. It's not ideal but moving away from Jekyll would be a considerable time investment and I have not yet found the time to do so.

@szarnyasg szarnyasg merged commit 5742ccb into duckdb:main Oct 31, 2025
4 checks passed
@Ian-Fogelman
Copy link
Contributor Author

Thanks!

Also just a note, was having a hard time to load these changes in the localhost browser, it took several restarts to see the changes in the railroad diagram. Potential caching issue/dynamic build issue?

Hi, yes, Jekyll's incremental builds are not great. I usually just do git add on whatever I'm working on, then git clean and a full rebuild. It's not ideal but moving away from Jekyll would be a considerable time investment and I have not yet found the time to do so.

Nice I added #6117 to the contributing.md to capture this tip!

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants