gortiz commented on PR #14037: URL: https://github.com/apache/pinot/pull/14037#issuecomment-2363644952
> There is one more in pinot-timeseries-spi as ../../pom.xml. Is that correct? It looks odd. I don't know why @ankitsultana used that reference. Anyway, that value may be semantically correct or not, but it is not the same as the ones I'm removing here that don't reference a `pom.xml` (but a folder) which is _syntactically_ incorrect (and the semantic they want to implement is the one defined by default by Maven) -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
