Use Logical type instead type in generate

[andrehinckel]

I've a question, There's a way to use the Logical type in generator? I've this schema

{
	"name": "updatedAt",
	"type": [
		"null",
		{
			"type": "int",
			"logicalType": "date"
		}
	]
}

and I'd like to use the date instead the int type.

Obs: Using the oficial lib of Avro I got this

and got this serialization result

Using the Chr.Avro I got this serialization result

[dstelljes]

Chr.Avro supports mapping the "timestamp-millis" and "timestamp-micros" logical types to DateTime, so you could use one of those. It doesn’t support mapping "date" to DateTime due to the lack of a time component (see the docs).

To customize your generated schemas, you can use the --temporal-behavior flag on the CLI or the temporalBehavior parameter on SchemaBuilder.

We’d be open to supporting "date" with the new DateOnly type, but we don’t have that functionality presently.

[tkring]

When I run the generator, it is not allowing me to pass the --temporal-behavior flag.

avro generate < c:\test.avsc > TestModel.cs --temporal-behavior epochmilliseconds
Chr.Avro.Cli 9.4.1+6cf08cdb2f191baa648a612b9f33efce01a5b7b4
Copyright (C) 2023 C.H. Robinson

ERROR(S):
Option 'temporal-behavior' is unknown.

When I run the generator without that flag against an avsc file with the following content, it generates a model with this property, Notice that the type is int instead of DateTime

public int? SomeTimestamp { get; set; }

avsc file content:
{
"type" : "record",
"namespace" : "TestNamespace",
"name" : "TestModel",
"fields" : [
{
"name" : "SomeTimestamp",
"type": [
"null",
{
"type": "int",
"logicalType": "timestamp-millis"
}
]
}
]
}

I'm using the main branch. Should I be using some other branch?

[dstelljes]

Try using "type": "long" instead of "type": "int" for your timestamp field. "timestamp-millis" only applies to longs.

[tkring]

Thanks! From: Dan Stelljes ***@***.***> Sent: Monday, April 17, 2023 6:03 AM To: ch-robinson/dotnet-avro ***@***.***> Cc: Kring, Tom ***@***.***>; Comment ***@***.***> Subject: [EXTERNAL] Re: [ch-robinson/dotnet-avro] Use Logical type instead type in generate (Discussion #231) Try using "type": "long" instead of "type": "int" for your timestamp field. "timestamp-millis" only applies to longs. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented. Message ZjQcmQRYFpfptBannerStart Warning! This email comes from outside the company. This message came from outside of Itron, if you suspect it to be spam, please delete it or if you suspect a phishing attempt, please report it using the Phish Alert Report button. Be careful and think before you click. ZjQcmQRYFpfptBannerEnd Try using "type": "long" instead of "type": "int" for your timestamp field. "timestamp-millis" only applies to longs. — Reply to this email directly, view it on GitHub<https://urldefense.com/v3/__https:/github.com/ch-robinson/dotnet-avro/discussions/231*discussioncomment-5636905__;Iw!!F7jv3iA!0OeGMXs-K_UnwuRMvpASefGQNXpmVHc7A1qfVqwGc3TsZi-6sXr-GfcSuSIa82mpkL3ZpQ8FACtkZ-jHXJL47jU$>, or unsubscribe<https://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AAJJQ23QEXMDDW6CYMQQLRLXBU5O3ANCNFSM6AAAAAAQSOIECE__;!!F7jv3iA!0OeGMXs-K_UnwuRMvpASefGQNXpmVHc7A1qfVqwGc3TsZi-6sXr-GfcSuSIa82mpkL3ZpQ8FACtkZ-jHUj5xRn8$>. You are receiving this because you commented.Message ID: ***@***.******@***.***>>