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

Rework command for adding a new source to an indexing #1270

Merged
merged 1 commit into from
Apr 11, 2022

Conversation

guilload
Copy link
Member

@guilload guilload commented Apr 8, 2022

Description

The new command is quickwit source create --index my-index --source-config my-source-config.yaml. As we may revisit how me manage objects in a near future (#1024), I chose to leave the index out of the config so I did not have to write some custom parsing logic.

Fixes #1071

How was this PR tested?

make test-all

@guilload guilload requested review from fmassot and fulmicoton April 8, 2022 14:13
@guilload guilload force-pushed the guilload--#1071--rework-source-add-command branch from 67dd3a8 to 34eb196 Compare April 8, 2022 14:13
Copy link
Contributor

@fulmicoton fulmicoton left a comment

Choose a reason for hiding this comment

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

I wasn't able to go too deep on how much of your refactoring was applied or useless.
I leave you judge to what to do there. If there is a follow-up needed, please create a ticket so we do not forget.

@guilload guilload force-pushed the guilload--#1071--rework-source-add-command branch from 34eb196 to 9e66ae1 Compare April 11, 2022 15:29
Copy link
Contributor

@fmassot fmassot left a comment

Choose a reason for hiding this comment

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

LGTM

@guilload guilload merged commit 9d42551 into main Apr 11, 2022
@guilload guilload deleted the guilload--#1071--rework-source-add-command branch April 11, 2022 15:59
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.

Add source CLI experience inconsistent with add an index [Papercut]
3 participants