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

Cross-link #284

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Cross-link #284

wants to merge 1 commit into from

Conversation

mitar
Copy link
Contributor

@mitar mitar commented Nov 7, 2021

No description provided.

@brandur
Copy link
Owner

brandur commented Nov 7, 2021

Thanks for the patch, but hmm, I don't think these are quite related enough for this link to be needed — there's lots of other info on Postgres atomicity out there, and someone reading this is likely to be at least somewhat aware of how atomicity works already.

@mitar
Copy link
Contributor Author

mitar commented Nov 7, 2021

As you want. I just think if you see all your writings as something one would publish in a book, it is nice to say something like "we will not go into how this works in this chapter, but read chapter X". Especially given that you have written the chapter X, too. Others might have written it to, but readers of your book might want to read your chapter X. And you can always then link to what others have written from chapter X for further reading.

@brandur
Copy link
Owner

brandur commented Nov 8, 2021

Ah gotcha. Yeah that does make some amount of sense, but these would definitely need a heavy hand in re-organization to get together a good overall narrative.

@mitar
Copy link
Contributor Author

mitar commented Nov 8, 2021

Given that I have read most of your blog in the past few days :-) I think it is already amazingly well organized with many cross-references already. This one is not there because the blog posts were written in the opposite order, so you could not made the link into the future. :-)

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