akupchinskiy commented on PR #2071: URL: https://github.com/apache/datafusion-comet/pull/2071#issuecomment-3156316423
> Thanks @akupchinskiy this change probably is too hasty, native readers like `datafusion` or `native_compat` are still experimental > > UPD: `native_datafusion` still experimental and `native_iceberg_compat` is not. It would probably nice to have another section in the doc explaining how to enable iceberg reader for the stability plans? @comphead It is not about experimental readers - currently plan stability suite works only with native reader . The intention was aligning the documentation with the actual ci/cd workflow. Without this change, the suite executes with no errors because there are no tests run at all - since it [expects the **native_comet** scanner set explicitly instead of default **auto** value](https://github.com/apache/datafusion-comet/blob/a583db3b94bc3ef36540d6804e1830a57fdb9f96/spark/src/test/scala/org/apache/spark/sql/comet/CometPlanStabilitySuite.scala#L321). -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
