Skip to content
New issue

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

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

Already on GitHub? # to your account

shared-db #1717

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

shared-db #1717

wants to merge 95 commits into from

Conversation

DanSchlachter
Copy link
Contributor

No description provided.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Linting Errors

  • guides/deployment/shared-db.md:86 Fenced code blocks should have a language specified
  • guides/deployment/shared-db.md:186 Fenced code blocks should have a language specified
  • guides/deployment/shared-db.md:255 Fenced code blocks should have a language specified
  • guides/deployment/shared-db.md:415 Fenced code blocks should have a language specified
  • guides/deployment/shared-db.md:423 Fenced code blocks should have a language specified

Spelling Mistakes

  • guides/deployment/shared-db.md:13:28 Unknown word "microserivces"
  • guides/deployment/shared-db.md:38:40 Unknown word "Clound"
  • guides/deployment/shared-db.md:56:30 Unknown word "esample"
  • guides/deployment/shared-db.md:247:3 Unknown word "Authentification"
  • guides/deployment/shared-db.md:287:43 Unknown word "asynchronious"
  • guides/deployment/shared-db.md:384:4 Unknown word "authentification"
  • guides/deployment/shared-db.md:398:29 Unknown word "earch"
  • guides/deployment/shared-db.md:439:23 Unknown word "authentification"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Linting Errors

  • guides/deployment/shared-db.md:90 Fenced code blocks should have a language specified
  • guides/deployment/shared-db.md:190 Fenced code blocks should have a language specified
  • guides/deployment/shared-db.md:259 Fenced code blocks should have a language specified
  • guides/deployment/shared-db.md:419 Fenced code blocks should have a language specified
  • guides/deployment/shared-db.md:427 Fenced code blocks should have a language specified

Spelling Mistakes

  • guides/deployment/shared-db.md:42:40 Unknown word "Clound"
  • guides/deployment/shared-db.md:60:30 Unknown word "esample"
  • guides/deployment/shared-db.md:251:3 Unknown word "Authentification"
  • guides/deployment/shared-db.md:291:43 Unknown word "asynchronious"
  • guides/deployment/shared-db.md:388:4 Unknown word "authentification"
  • guides/deployment/shared-db.md:402:29 Unknown word "earch"
  • guides/deployment/shared-db.md:443:23 Unknown word "authentification"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Linting Errors

  • guides/deployment/shared-db.md:271 Fenced code blocks should have a language specified

Spelling Mistakes

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Spelling Mistakes

  • guides/deployment/shared-db.md:42:257 Unknown word "Addinital"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Spelling Mistakes

  • guides/deployment/late-cut-microservices.md:55:5 Unknown word "Modulith"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

With that we're basically done with the setup of the collector project. At the end of the day, it's just another CAP project with some cds models in it, which we can handle as usual. We can test whether it all works as expected, for example, we can test-compile and test-deploy it to sqlite and hana, build it, and deploy it to the cloud as usual:

```sh
cds db -2 sql
Copy link
Contributor

Choose a reason for hiding this comment

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

What is cds db about? And why is this step needed?

Copy link
Contributor

Choose a reason for hiding this comment

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

This step is not needed -> it's just to show that it works just like you would use it in a standalone cap project... since these are not necessary steps, we put it inside a details block

Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks.
I've got cds-dk@8.8.1 and that doesn't know cds db. What does it do?


Assumed we want to create a composite application consisting of two or more micro services, each living in a separate GitHub repository, for example:

- https://github.com/capire/bookstore
Copy link
Contributor Author

Choose a reason for hiding this comment

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

None existing links - will be picked up by broken link checker?
Repos don't exist or are outdated - isn't this confusing?
Maybe start with cap-samples directly and add a details section for setting up a new project?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, either create the repos or describe some other way.
I think it's good to have this front and center (monorepo with submodules) as it would be the typical project setup.

We could have a script that copies the folders from cloud-cap-samples and keeps them up to date...

Copy link
Contributor

Choose a reason for hiding this comment

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

Ideally there'd be a command like

git submodule add https://github.com/SAP-samples/cloud-cap-samples --folder bookstore

But there isn't as far as I'm aware. There seems to be some way to achieve something similar, but too complicated for a guide :D

:::


### Deployment as separate mta
Copy link
Contributor Author

Choose a reason for hiding this comment

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

shouldn't this be on the same heading level as All-in-one Deployment
Can we move it to the end of the guide and clearly mention, that it is an alternative approach? It's unclear that it's either one or the other

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes, my thoughts exactly - been wondering where it would be a good fit, let's do it like that

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Spelling Mistakes

  • guides/deployment/microservices.md:85:13 Unknown word "commited"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Spelling Mistakes

  • guides/deployment/microservices.md:85:13 Unknown word "commited"

Generally, for each spelling mistake there are 2 ways to fix it:

  1. Fix the spelling mistake and commit it.
  2. The word is incorrectly reported as misspelled → put the word on the project-words.txt list, located in the root project directory.

@Akatuoro Akatuoro requested a review from renejeglinsky April 8, 2025 12:52
@Akatuoro Akatuoro enabled auto-merge April 8, 2025 12:53
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants