[
https://issues.apache.org/jira/browse/AVRO-3547?focusedWorklogId=792570&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-792570
]
ASF GitHub Bot logged work on AVRO-3547:
----------------------------------------
Author: ASF GitHub Bot
Created on: 19/Jul/22 10:35
Start Date: 19/Jul/22 10:35
Worklog Time Spent: 10m
Work Description: KalleOlaviNiemitalo commented on code in PR #1736:
URL: https://github.com/apache/avro/pull/1736#discussion_r924144410
##########
lang/c++/impl/Compiler.cc:
##########
@@ -146,7 +149,8 @@ struct Field {
const string name;
const NodePtr schema;
const GenericDatum defaultValue;
- Field(string n, NodePtr v, GenericDatum dv) : name(std::move(n)),
schema(std::move(v)), defaultValue(std::move(dv)) {}
+ const CustomFields customFields;
+ Field(string n, NodePtr v, GenericDatum dv, const CustomFields& cf) :
name(std::move(n)), schema(std::move(v)), defaultValue(std::move(dv)),
customFields(std::move(cf)) {}
Review Comment:
std::move(cf) here seems odd because cf is a const&. But I guess it'll just
make a const&& and not cause any problem.
Issue Time Tracking
-------------------
Worklog Id: (was: 792570)
Time Spent: 2h 20m (was: 2h 10m)
> support custom attribute at field level
> ---------------------------------------
>
> Key: AVRO-3547
> URL: https://issues.apache.org/jira/browse/AVRO-3547
> Project: Apache Avro
> Issue Type: Improvement
> Components: c++
> Reporter: Nilesh Yadav
> Priority: Major
> Labels: pull-request-available
> Time Spent: 2h 20m
> Remaining Estimate: 0h
>
> Custom attributes are allowed at field level as per Avro specification :
> [https://avro.apache.org/docs/current/spec.html#schemas|https://www.google.com/url?q=https://avro.apache.org/docs/current/spec.html%23schemas&sa=D&source=buganizer&usg=AOvVaw3JfXMgFX6Eq0Fd0TjDzgb1]
>
> But Avro c++ implementation does not support custom attribute at field level.
> e.g. following schema cannot be parsed correctly due to presence of "sqlName"
> attribute.
> {
> 'doc': 'weather warning',
> 'name': 'weather',
> 'type': 'record',
> 'fields': [
> {
> 'name': 'station',
> 'type':'string',
> 'sqlName':'abc-@12'
> },
> {
> 'name': 'time',
> 'type':'long',
> 'sqlName': '用户名'
> }]
> }
> Update implementation to support custom attribute at field level.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
