apache / apache/datafusion-python

Remove all uses of `todo!`

Fermée
#552 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
enhancement
Langage dominant
Python
Étoiles
604
Forks
174
Merge moyen
1 j 7 h
PR mergées (30 j)
4

Description

**Is your feature request related to a problem or challenge? Please describe what you are trying to do.**

We have a number of uses of `todo!` that we should replace with use of `Err`.

```
$ find . -name "*.rs" -exec grep -H "todo\!" {} \;
./src/expr/indexed_field.rs: _ => todo!(),
./src/expr.rs: ScalarValue::Null => todo!(),
./src/expr.rs: ScalarValue::Decimal256(_, _, _) => todo!(),
./src/expr.rs: ScalarValue::FixedSizeBinary(_, _) => todo!(),
./src/expr.rs: ScalarValue::List(_) => todo!(),
./src/expr.rs: ScalarValue::Struct(_, _) => todo!(),
./src/expr.rs: ScalarValue::Dictionary(_, _) => todo!(),
./src/expr.rs: ScalarValue::FixedSizeList(_) => todo!(),
./src/expr.rs: ScalarValue::LargeList(_) => todo!(),
./src/expr.rs: Operator::AtArrow | Operator::ArrowAt => todo!(),
./src/common/data_type.rs: todo!()
./src/common/data_type.rs: ScalarValue::LargeList(_) => todo!(),
```

**Describe the solution you'd like**

**Describe alternatives you've considered**

**Additional context**

Guide de contribution

Aucun guide de contribution indexé pour ce dépôt

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.