Reactivesearch: app/appName is too oriented appbaseio products

Created on 11 Jan 2018  路  4Comments  路  Source: appbaseio/reactivesearch

Issue Type:
enhancement

Description:
You use the value/prop app or appName in your code or doc but it's clearly too oriented to your company projects.

I had tough time to understand what it is, and it should be called indices like it's called in ES doc. https://www.elastic.co/guide/en/elasticsearch/reference/current/multi-index.html

Thank you.

Most helpful comment

No worries, I understand. ;) It's just I took time to understand what means appName in our own ES. :)

Maybe both could/should be accepted? or maybe a note in the doc (appName means indices)

All 4 comments

Good point, thanks for the feedback.

We have built this library also keeping in mind, the developers who are not so familiar with Elasticsearch ecosystem, and since the library is fully compatible with appbase.io where we don't expose the concept of indices, it made sense for us to use app or appName as the prop name. We will update our docs to cover this surely.

Although, it's a good point, having indices as the prop-name improves the experience for developers familiar with Elasticsearch.

Thoughts @siddharthlatest?

No worries, I understand. ;) It's just I took time to understand what means appName in our own ES. :)

Maybe both could/should be accepted? or maybe a note in the doc (appName means indices)

Haaaa amazing! Thank you @divyanshu013 !

Was this page helpful?
0 / 5 - 0 ratings

Related issues

kud picture kud  路  4Comments

willopez picture willopez  路  3Comments

scheiblr picture scheiblr  路  3Comments

danielsnider picture danielsnider  路  4Comments

gabzon picture gabzon  路  4Comments