search

nck-2 / test-rep

0 stars 0 forks source link

change of index type #1276

Closed githubmanticore closed 1 year ago

githubmanticore commented 1 year ago

daemon might be able to change index type on INDEXES RELOAD or SIGHUP command when index type changed at config.

However that does not work for RT and PQ index when path is same as daemon does not clean up previous index files after index got deleted and try to load new index with existed path then failed with error message: 

WARNING: index 'RT_PQ_idx': prealloc: invalid meta file C:/dev/sphinx/manticore/build/test/data/rt_pq.meta; NOT SERVING

it might be better to add additional logic that prohibit save of previous index content, cleans its files then loads new index.

githubmanticore commented 1 year ago

➤ Sergey Nikolaev commented:

daemon might be able to change index type on INDEXES RELOAD or SIGHUP command when index type changed at config.

Why is it important? Why do we need to support it?

githubmanticore commented 1 year ago

➤ Stan commented:

as @klirichek said he made it to promote local \ RT \ PQ index to distributed. However for now code made this way that RT might be changed to PQ and vice versa there is no restriction and might cause issue initially described