You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We can't do this right away because the Iceberg integration depends on native_comet for now, but we should start working towards removing the native_comet scan implementation.
Motivation:
Reduce the amount of code to maintain by reducing the number of scan implementations from 3 to 2
The first step will be to review all of the tests where we currently explicitly set the scan implementation to native_comet. For each test we need to consider:
Can we update the test such that it passes when using the default scan implementation?
Is the test specific to native_comet and can therefore just be removed (when we stop supporting native_comet)?
Does the test fail with the default scan because there is a bug or missing feature? If so, can we just fall back to Spark?
We need to implement the following before removing native_comet:
Activity
andygrove commentedon Feb 5, 2026
native_cometscan was removed in #3358