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

Change some leftover references from external data store to data connection #1251

Merged

Conversation

frant-hartm
Copy link
Contributor

Data connection feature was called external data store in its beta phase (in Hazelcast 5.3). Apparently, there are still some references to the old name left.

I have tried to capture everything, apart from the names of the pages / links - I don't understand how the redirects work exactly and didn't want to break anything.

…tion

Data connection feature was called external data store in its beta phase
(in Hazelcast 5.3). Apparently there are still some references to the
old name left. I have tried to capture everything, apart from the names
of the pages / links - I don't understand how the redirects work exactly
and didn't want to break anything.
Copy link

netlify bot commented Aug 12, 2024

Deploy Preview for hardcore-allen-f5257d ready!

Name Link
🔨 Latest commit 903c2b7
🔍 Latest deploy log https://app.netlify.com/sites/hardcore-allen-f5257d/deploys/672e4ba85776240007cbcbf9
😎 Deploy Preview https://deploy-preview-1251--hardcore-allen-f5257d.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.

Copy link
Contributor

@oliverhowell oliverhowell left a comment

Choose a reason for hiding this comment

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

Minor comments but approving in advance

@@ -45,11 +45,11 @@ p.readFrom(Sources.jdbc(
)).writeTo(Sinks.logger());
```

You can also use a configured xref:external-data-stores:external-data-stores.adoc#defining-external-data-stores[external data store] as a JDBC Source:
You can also use a configured xref:external-data-stores:external-data-stores.adoc#defining-external-data-stores[data connection] as a JDBC Source:
Copy link
Contributor

Choose a reason for hiding this comment

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

This page doesn't exist...looks like data-connections-configuration.adoc has the alias and the link should still work

Copy link
Contributor

Choose a reason for hiding this comment

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

The link does still work, but the updated link text wasn't showing up in the build for some reason. I've just updated all these xrefs (and any child section refs) to point to the new file.

docs/modules/sql/pages/mapping-to-jdbc.adoc Outdated Show resolved Hide resolved
@Rob-Hazelcast Rob-Hazelcast merged commit f4e4b21 into hazelcast:main Nov 11, 2024
5 of 6 checks passed
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.

4 participants