-
Notifications
You must be signed in to change notification settings - Fork 369
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
Shall we rename the Indexer component ? #21
Comments
It's been in the todo to rename it to I would prefer to keep it 1-word for simplicity, what about |
Well, I believe that |
It will be used to name the service in bicep and yaml too, and I think using more than 1 word might be confusing, ie What bother me is that we called the process "data ingestion" here, but what we're doing technically is indexing data into the DB (that's how it's called too in Azure docs). Maybe the good solution is to keep the |
Cultural difference between JS and Java developers. We love our long class names ;o) |
I quite like the 1-word version: |
I agree here, the way I understood the typescript code is that you're doing indexing here. Copilot answer: Data ingestion and indexing are both crucial steps in managing and utilizing data, but they serve different purposes:
In summary, data ingestion focuses on getting data into the system, while indexing optimizes data access and retrieval. Both processes are essential for effective data management and analysis. |
@sinedied @SandraAhlgrimm so, shall we rename +1 for me |
I've renamed |
At first I didn't understand what "Indexer" meant in the document processor. And the documentation mentions "Data ingestion". So what about renaming the component "Indexer" to something else:
data-ingestion
data-ingestor
ingestor
(does it exist in Engligh or is itingester
)The text was updated successfully, but these errors were encountered: