bug: dup batch is returned repeatedly if set filter_expr on legacy dataset with stable row ID enabled

[BubbleCal]

with data storage version Legacy and stable row ID enabled, when try scanning the dataset with filter_expr, the same batches are returned repeatedly

reproduce steps:

    #[tokio::test]
    async fn test_stable_row_id() {
        let test_ds = TestVectorDataset::new(LanceFileVersion::Legacy, true)
            .await
            .unwrap();
        let dataset = &test_ds.dataset;
        let mut data = dataset
            .scan()
            .batch_readahead(get_num_compute_intensive_cpus())
            .project(&["vec"])
            .unwrap()
            .with_row_id()
            .filter_expr(datafusion_expr::col("vec").is_not_null())
            .try_into_stream()
            .await
            .unwrap();

        let mut row_id_set = BTreeSet::new();
        while let Some(batch) = data.try_next().await.unwrap() {
            let row_ids = batch[ROW_ID].as_primitive::<UInt64Type>();
            row_ids.values().iter().for_each(|v| {
                assert!(row_id_set.insert(*v), "dup row id: {}", v);
            });
        }
    }