FrankChen021 commented on issue #20682: URL: https://github.com/apache/druid/issues/20682#issuecomment-6331958370
@BackendArchitectX @clintropolis looping you in, since this comes from #20633 and #20617 landing about an hour apart. #20633 added `Projection[%s] does not exist` to `Projections.findMatchingProjection`. `PartialQueryableIndex.getServableProjection` passes that method only the servable projections, so a projection that is in the segment but not servable (`dim1_count` in `testUseProjectionNamingUnservedProjectionFails`) is now reported as not existing. The test's expectation (`does not satisfy query`) looks right to us, so the proposed follow-up is to run the existence check against the full `projections` set and keep `does not exist` for names the segment doesn't have. Does that match what each of you intended? -- 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]
