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

LC-202 Backing up multiple topics at once #1220

Merged
merged 7 commits into from
May 14, 2024

Conversation

stheppi
Copy link
Contributor

@stheppi stheppi commented May 13, 2024

To back the topics up the KCQL statemet is

INSERT INTO bucket
SELECT * FROM `*`
...

When * is used the envelope setting is ignored.

This change allows for the * to be taken into account as a default if the given message topic is not found.

@stheppi stheppi marked this pull request as ready for review May 13, 2024 16:18
stheppi added 4 commits May 13, 2024 23:06
To back the topics up the KCQL statemet is

```
INSERT INTO bucket
SELECT * FROM `*`
...
```

When `*` is used the envelope setting is ignored.

This change allows for the `*` to be taken into account as a default if the given message topic is not found.
@stheppi stheppi force-pushed the fix/LC-202-star-does-not-store-envelope branch from 12e6a6f to 209b47e Compare May 13, 2024 22:10
@stheppi stheppi merged commit d0c1c24 into master May 14, 2024
161 checks passed
@stheppi stheppi deleted the fix/LC-202-star-does-not-store-envelope branch May 14, 2024 19:30
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