birschick-bq commented on code in PR #3551:
URL: https://github.com/apache/arrow-adbc/pull/3551#discussion_r2415472092


##########
csharp/src/Apache.Arrow.Adbc/Apache.Arrow.Adbc.csproj:
##########
@@ -10,7 +10,7 @@
     <PackageReference Include="System.Diagnostics.DiagnosticSource" 
Version="9.0.6" />
   </ItemGroup>
   <ItemGroup 
Condition="!$([MSBuild]::IsTargetFrameworkCompatible($(TargetFramework), 
'net6.0'))">
-    <PackageReference Include="System.Text.Json" Version="8.0.5" />
+    <PackageReference Include="System.Text.Json" Version="9.0.0" />

Review Comment:
   I updated the Apache.Thrift to version 0.22.0 because I needed the changes I 
made to set the InnerException on TTransportExeptions. Otherwise, I would just 
have to look at the text of the exception message.
   
   Well to use Thrift 0.22.0, it updates the Microsoft.Extension.Loggin.Console 
to >= 9.0.0 which in turn requires System.Text.JSon >= 9.0.0
   
   https://www.nuget.org/packages/ApacheThrift/0.22.0#dependencies-body-tab
   
   I could not find a way to keep System.Text.Json to 8.0.5



-- 
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]

Reply via email to