[
https://issues.apache.org/jira/browse/AVRO-3388?focusedWorklogId=727846&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-727846
]
ASF GitHub Bot logged work on AVRO-3388:
----------------------------------------
Author: ASF GitHub Bot
Created on: 15/Feb/22 21:11
Start Date: 15/Feb/22 21:11
Worklog Time Spent: 10m
Work Description: zcsizmadia commented on a change in pull request #1536:
URL: https://github.com/apache/avro/pull/1536#discussion_r807318209
##########
File path: lang/csharp/build.sh
##########
@@ -38,6 +38,10 @@ do
# AVRO-2442: Explicitly set LANG to work around ICU bug in `dotnet test`
LANG=en_US.UTF-8 dotnet test --configuration Release --no-build \
--filter "TestCategory!=Interop" Avro.sln
+
+ dotnet build --configuration Release src/apache/codec/Avro.Codec.sln
+ ANG=en_US.UTF-8 dotnet test --configuration Release --no-build \
Review comment:
I really just added this to be able to test the codecs by gitflow. At
this stage I was not even sure that this Codec lib package should be part of
this repo or a sepearet repo?
Other idea I had was that Avro.main C# has all the defines as well for all
the codecs just like Java, however in the CreateCodec(...) it creates an codec
from :Apache.Avro.Codec....." assembly.
If the project was built with PackageReference Apache.Avro.Codec.Snappy, the
snappy codec will be available, otherwise unknwo codec exception.
So I had many questions here, at this stage it was jsut a proof of concept
to have seperate nuget packages fopr all supported codecs, and how others like
the idea. I like this way sure, since Avro.main does not have any new package
dependencies. The application using the C# cllibrary will have to add the
codecs which it wants to use. All of them if it wants to support all.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 727846)
Time Spent: 4h 10m (was: 4h)
> Implement extra codecs for C# as seperate nuget packages
> --------------------------------------------------------
>
> Key: AVRO-3388
> URL: https://issues.apache.org/jira/browse/AVRO-3388
> Project: Apache Avro
> Issue Type: New Feature
> Components: csharp
> Reporter: Zoltan Csizmadia
> Priority: Minor
> Labels: pull-request-available
> Time Spent: 4h 10m
> Remaining Estimate: 0h
>
> C# main library supports only Deflate and Null codecs. There are other
> supported codecs using Java: XZ, bzip2, zstandard, snappy.
>
> C# has support for all these codecs, however they require many dependencies.
> It makes sense to create nuget packages for each codec, and the end user
> references the codecs, which are needed.
--
This message was sent by Atlassian Jira
(v8.20.1#820001)
