pinot-contrib / pinot-docs

Apache Pinot Documentation
23 stars 156 forks source link

Index spi #162

Closed gortiz closed 10 months ago

gortiz commented 1 year ago

Large changes to:

kelseiv commented 1 year ago

to be sure, @gortiz, did you mean to delete several properties in the Table config section? (looking at the unresolved conflicts) --thank you!

gortiz commented 1 year ago

to be sure, @gortiz, did you mean to delete several properties in the Table config section? (looking at the unresolved conflicts) --thank you!

Yes. These properties are still valid, but not recommended to be set. Instead, the fieldConfigList.indexes section should be used. Given that there are some properties that are still needed (the ones I didn't remove) and that the property table is quite large, I think it is better to just remove the properties that users should stop using. All the removed properties are also explained on each specific index description, so the information won't be lost. I removed them from the table in order to make this section (table.md) easier to read.

gortiz commented 12 months ago

@kelseiv I've applied tons of style changes in the text. Feel free to review it whenever you can