chore: adjust fuse_time_travel_size()

[SkyFan2002]

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

1. In the past, fuse_time_travel_size() did not account for tables with a drop time exceeding 24 hours. After this PR, these tables will be included in the count.
2. This PR will not affect the behavior of other parts of the system.

Tests

• Unit Test
• Logic Test
• Benchmark Test
• No Test - Explain why

Type of change

• Bug Fix (non-breaking change which fixes an issue)
• New Feature (non-breaking change which adds functionality)
• Breaking Change (fix or feature that could cause existing functionality not to work as expected)
• Documentation Update
• Refactoring
• Performance Improvement
• Other (please describe):

---

This change is 

[drmingdrmer]

A test should be added to ensure non-retainable tables are returned as expected

Reviewed 4 of 7 files at r1, all commit messages.
Reviewable status: 4 of 7 files reviewed, 4 unresolved discussions (waiting on @dantengsky, @SkyFan2002, and @TCeason)

---

src/meta/api/src/schema_api.rs line 230 at r1 (raw file):

        include_non_retainable: bool,
        req: ListTableReq,
    ) -> Result<Vec<TableNIV>, KVAppError>;

The method name should be updated too, if it returns both retainable and non-retainable tables.

Code quote:

    async fn list_retainable_tables(
        &self,
        include_non_retainable: bool,
        req: ListTableReq,
    ) -> Result<Vec<TableNIV>, KVAppError>;

---

src/meta/api/src/schema_api_impl.rs line 3068 at r1 (raw file):

async fn get_retainable_table_metas(
    kv_api: &(impl kvapi::KVApi<Error = MetaError> + ?Sized),
    include_non_retainable: bool,

Update method name

Code quote:

async fn get_retainable_table_metas(
    kv_api: &(impl kvapi::KVApi<Error = MetaError> + ?Sized),
    include_non_retainable: bool,

---

src/query/service/src/databases/default/default_database.rs line 199 at r1 (raw file):

        &self,
        include_non_retainable: bool,
    ) -> Result<Vec<Arc<dyn Table>>> {

update method name

Code quote:

    async fn list_tables_history(
        &self,
        include_non_retainable: bool,
    ) -> Result<Vec<Arc<dyn Table>>> {

---

src/query/service/src/databases/default/default_database.rs line 211 at r1 (raw file):

                include_non_retainable,
                ListTableReq::new(self.get_tenant(), self.db_info.database_id),
            )

update method name

Code quote:

            .list_retainable_tables(
                include_non_retainable,
                ListTableReq::new(self.get_tenant(), self.db_info.database_id),
            )

[drmingdrmer]

Reviewed 2 of 7 files at r1, 4 of 4 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @dantengsky and @TCeason)

[TCeason]

LGTM this logic not impact db and tables with history.