Skip to content

Commit

Permalink
Merge pull request #118 from lr4d/remove_unnecesary_code
Browse files Browse the repository at this point in the history
remove unnecesary code
  • Loading branch information
fjetter authored Aug 15, 2019
2 parents c199c10 + 673c79a commit 37ded7b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
2 changes: 0 additions & 2 deletions kartothek/serialization/_generic.py
Original file line number Diff line number Diff line change
Expand Up @@ -268,8 +268,6 @@ def _ensure_type_stability(array_like, value, strict_date_types, require_ordered
array_like, array_value_type = _handle_categorical_data(array_like, require_ordered)
array_like, array_value_type = _handle_null_arrays(array_like, value_dtype)

type_comp = (value_dtype.kind, array_value_type.kind)

compatible_types = [
# UINT and INT
("u", "i"),
Expand Down
5 changes: 1 addition & 4 deletions kartothek/serialization/testing.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,10 +7,7 @@


def _to_binary(x):
x = str(x)
if isinstance(x, str):
x = x.encode("utf-8")
return x
return str(x).encode("utf-8")


BINARY_COLUMNS = [
Expand Down

0 comments on commit 37ded7b

Please sign in to comment.