Skip to content

Commit 9761da4

Browse files
committed
Pin the nightly version to avoid slipping linting and compiler errors
1 parent 82aabdf commit 9761da4

3 files changed

Lines changed: 9 additions & 7 deletions

File tree

.github/workflows/ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ jobs:
4545
- name: Install minimal nightly with clippy and rustfmt
4646
uses: dtolnay/rust-toolchain@master
4747
with:
48-
toolchain: nightly
48+
toolchain: nightly-2024-12-30
4949
components: rustfmt, clippy
5050

5151
- name: Install tool for formatting Cargo.toml files

.github/workflows/nightly.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,8 @@ jobs:
4343
- name: Checkout the repository
4444
uses: actions/checkout@v4
4545
- run: |
46-
rustup toolchain install nightly --profile minimal
47-
rustup default nightly
46+
rustup toolchain install nightly-2024-12-30 --profile minimal
47+
rustup default nightly-2024-12-30
4848
4949
- uses: Swatinem/rust-cache@v2
5050
with:

src/context/mod.rs

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -152,7 +152,8 @@ impl SeafowlContext {
152152
&self,
153153
table_name: impl Into<TableReference>,
154154
) -> Result<LakehouseTableProvider> {
155-
let table_provider = self.inner.table_provider(table_name).await?;
155+
let table_ref = table_name.into();
156+
let table_provider = self.inner.table_provider(table_ref.clone()).await?;
156157
let table_provider_any = table_provider.as_any();
157158

158159
if let Some(a) = table_provider_any.downcast_ref::<DeltaTable>() {
@@ -163,7 +164,7 @@ impl SeafowlContext {
163164
}
164165

165166
Err(DataFusionError::Execution(
166-
"Table {table_name} not found".to_string(),
167+
format!("Table {table_ref} not found").to_string(),
167168
))
168169
}
169170

@@ -172,10 +173,11 @@ impl SeafowlContext {
172173
&self,
173174
table_name: impl Into<TableReference>,
174175
) -> Result<DeltaTable> {
175-
match self.get_lakehouse_table_provider(table_name).await? {
176+
let table_ref = table_name.into();
177+
match self.get_lakehouse_table_provider(table_ref.clone()).await? {
176178
LakehouseTableProvider::Delta(delta_table) => Ok(delta_table),
177179
_ => Err(DataFusionError::Execution(
178-
"Delta table {table_name} not found".to_string(),
180+
format!("Delta table {table_ref} not found").to_string(),
179181
)),
180182
}
181183
}

0 commit comments

Comments
 (0)