You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@avro.apache.org by "Sahil Kang (Jira)" <ji...@apache.org> on 2024/03/10 23:20:00 UTC
[jira] [Created] (AVRO-3961) Add AVRO_INVALID to avro_type_t
Sahil Kang created AVRO-3961:
--------------------------------
Summary: Add AVRO_INVALID to avro_type_t
Key: AVRO-3961
URL: https://issues.apache.org/jira/browse/AVRO-3961
Project: Apache Avro
Issue Type: Improvement
Components: c
Reporter: Sahil Kang
There are preexisting warnings in [schema.c|https://github.com/apache/avro/blob/9c7e14d89f80f7bea9c4d67d3ae98b86b5cff166/lang/c/src/schema.c#L879-L886] and [datum_value.c|https://github.com/apache/avro/blob/9c7e14d89f80f7bea9c4d67d3ae98b86b5cff166/lang/c/src/datum_value.c#L83-L94] stemming from the fact that [avro_type_t|https://github.com/apache/avro/blob/9c7e14d89f80f7bea9c4d67d3ae98b86b5cff166/lang/c/src/avro/basics.h#L28-L44] does not have a default/invalid state
However, since the public interface already exposes [EINVAL as an avro_type_t|https://github.com/apache/avro/blob/9c7e14d89f80f7bea9c4d67d3ae98b86b5cff166/lang/c/src/datum_value.c#L95], we can add an {{AVRO_INVALID = EINVAL}} branch to {{avro_type_t}} while maintaining ABI compatibility
--
This message was sent by Atlassian Jira
(v8.20.10#820010)