You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by ba...@apache.org on 2023/06/06 18:17:35 UTC
[arrow-julia] branch main updated: #453 fix typeo introduced in #389 (#454)
This is an automated email from the ASF dual-hosted git repository.
baumgold pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-julia.git
The following commit(s) were added to refs/heads/main by this push:
new 7044e19 #453 fix typeo introduced in #389 (#454)
7044e19 is described below
commit 7044e19849382129e0712133ee5ebb13310f53ee
Author: Ben Baumgold <49...@users.noreply.github.com>
AuthorDate: Tue Jun 6 14:17:28 2023 -0400
#453 fix typeo introduced in #389 (#454)
I introduced this bug in #389 - my apologies. Fixes #453.
---
src/ArrowTypes/src/ArrowTypes.jl | 4 ++--
src/ArrowTypes/test/tests.jl | 2 ++
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/src/ArrowTypes/src/ArrowTypes.jl b/src/ArrowTypes/src/ArrowTypes.jl
index 6e77c1c..6dee157 100644
--- a/src/ArrowTypes/src/ArrowTypes.jl
+++ b/src/ArrowTypes/src/ArrowTypes.jl
@@ -119,8 +119,8 @@ This will then be available to access by overloading `ArrowTypes.JuliaType(::Val
function arrowmetadata end
const EMPTY_STRING = ""
arrowmetadata(T) = EMPTY_STRING
-arrowmetadata(::Type{Union{T,Missing}}) where {T} = arrowmetadata{T}
-arrowmetadata(::Type{Union{T,Nothing}}) where {T} = arrowmetadata{T}
+arrowmetadata(::Type{Union{T,Missing}}) where {T} = arrowmetadata(T)
+arrowmetadata(::Type{Union{T,Nothing}}) where {T} = arrowmetadata(T)
arrowmetadata(::Type{Nothing}) = EMPTY_STRING
arrowmetadata(::Type{Missing}) = EMPTY_STRING
arrowmetadata(::Type{Any}) = EMPTY_STRING
diff --git a/src/ArrowTypes/test/tests.jl b/src/ArrowTypes/test/tests.jl
index 26f3945..0c257dc 100644
--- a/src/ArrowTypes/test/tests.jl
+++ b/src/ArrowTypes/test/tests.jl
@@ -38,6 +38,8 @@ end
@test !ArrowTypes.hasarrowname(Int)
@test ArrowTypes.arrowmetadata(Int) == ""
+@test ArrowTypes.arrowmetadata(Union{Nothing,Int}) == ""
+@test ArrowTypes.arrowmetadata(Union{Missing,Int}) == ""
@test ArrowTypes.JuliaType(1) === nothing
@test ArrowTypes.JuliaType(1, Int) === nothing