raghavyadav01 opened a new pull request, #18930:
URL: https://github.com/apache/pinot/pull/18930

   The HNSW reader path from #18852 probes the local filesystem for legacy 
index files before reading the consolidated `columns.psf` entry. When the 
segment directory is not a local path (e.g. object-store backed), 
`findFormatFile`'s `checkArgument(isDirectory())` throws and the whole segment 
load fails (ERROR state).
   
   Fix: skip the probe when `segmentDir` is not a local directory — no legacy 
files can exist there. Both call sites in `VectorIndexType$ReaderFactory` (HNSW 
gate, IVF fallback).
   
   Testing: unit test for non-local path loading from the consolidated buffer; 
verified e2e on an S3-backed setup — segments ONLINE, `EXPLAIN` shows 
`ANN_TOP_K`.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to