Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MINOR: [C#] Bump Google.Protobuf from 3.25.0 to 3.25.1 in /csharp #38800

Merged
merged 1 commit into from
Nov 20, 2023
Merged
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -5,7 +5,7 @@
</PropertyGroup>

<ItemGroup>
<PackageReference Include="Google.Protobuf" Version="3.25.0" />
<PackageReference Include="Google.Protobuf" Version="3.25.1" />
<PackageReference Include="Grpc.Net.Client" Version="2.59.0" />
<PackageReference Include="Grpc.Tools" Version="2.59.0" PrivateAssets="All" />
</ItemGroup>

Unchanged files with check annotations Beta

_owner = owner;
}
~NativeMemoryManager()

Check warning on line 43 in csharp/src/Apache.Arrow/Memory/NativeMemoryManager.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Adding a finalizer to a type derived from MemoryManager<T> may permit memory to be freed while it is still in use by a Span<T> (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2015)

Check warning on line 43 in csharp/src/Apache.Arrow/Memory/NativeMemoryManager.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Adding a finalizer to a type derived from MemoryManager<T> may permit memory to be freed while it is still in use by a Span<T> (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2015)

Check warning on line 43 in csharp/src/Apache.Arrow/Memory/NativeMemoryManager.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Adding a finalizer to a type derived from MemoryManager<T> may permit memory to be freed while it is still in use by a Span<T> (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2015)

Check warning on line 43 in csharp/src/Apache.Arrow/Memory/NativeMemoryManager.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Adding a finalizer to a type derived from MemoryManager<T> may permit memory to be freed while it is still in use by a Span<T> (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2015)

Check warning on line 43 in csharp/src/Apache.Arrow/Memory/NativeMemoryManager.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Adding a finalizer to a type derived from MemoryManager<T> may permit memory to be freed while it is still in use by a Span<T> (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2015)

Check warning on line 43 in csharp/src/Apache.Arrow/Memory/NativeMemoryManager.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Adding a finalizer to a type derived from MemoryManager<T> may permit memory to be freed while it is still in use by a Span<T> (https://learn.microsoft.com/dotnet/fundamentals/code-analysis/quality-rules/ca2015)
{
Dispose(false);
}
var streamWriter = new MockServerStreamWriter<FlightActionType>();
//When
await producer.ListActions(streamWriter, new MockServerCallContext()).ConfigureAwait(false);

Check warning on line 68 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 68 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 68 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 68 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 68 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 68 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
var actions = streamWriter.Messages.ToArray();
Assert.Equal(FlightSqlUtils.FlightSqlActions, actions);
var producer = new TestFlightSqlSever();
var descriptor = FlightDescriptor.CreateCommandDescriptor(command.PackAndSerialize().ToArray());
var recordBatch = new RecordBatch(new Schema(new List<Field>(), null), System.Array.Empty<Int8Array>(), 0);
var reader = new MockStreamReader<FlightData>(await recordBatch.ToFlightData(descriptor).ConfigureAwait(false));

Check warning on line 247 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 247 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 247 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 247 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 247 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 247 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
var batchReader = new FlightServerRecordBatchStreamReader(reader);
var mockStreamWriter = new MockServerStreamWriter<FlightPutResult>();
//When
async Task Act() => await producer.DoPut(batchReader, mockStreamWriter, new MockServerCallContext()).ConfigureAwait(false);

Check warning on line 252 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 252 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 252 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 252 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 252 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
var exception = await Record.ExceptionAsync(Act);
string? actualMessage = isException ? exception?.Message : mockStreamWriter.Messages[0].ApplicationMetadata.ToStringUtf8();
protected override CancellationToken CancellationTokenCore => default;
protected override Metadata ResponseTrailersCore => new();
protected override Status StatusCore { get; set; }
protected override WriteOptions WriteOptionsCore { get; set; } = WriteOptions.Default;

Check warning on line 274 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Nullability of type of parameter 'value' doesn't match overridden member (possibly because of nullability attributes).

Check warning on line 274 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Nullability of type of parameter 'value' doesn't match overridden member (possibly because of nullability attributes).

Check warning on line 274 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Nullability of type of parameter 'value' doesn't match overridden member (possibly because of nullability attributes).

Check warning on line 274 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Nullability of type of parameter 'value' doesn't match overridden member (possibly because of nullability attributes).

Check warning on line 274 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Nullability of type of parameter 'value' doesn't match overridden member (possibly because of nullability attributes).

Check warning on line 274 in csharp/test/Apache.Arrow.Flight.Sql.Tests/FlightSqlServerTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Nullability of type of parameter 'value' doesn't match overridden member (possibly because of nullability attributes).
protected override AuthContext AuthContextCore => new("", new Dictionary<string, List<AuthProperty>>());
}
}
{
var duplexStreamingCall = _flightClient.Handshake();
await duplexStreamingCall.RequestStream.WriteAsync(new FlightHandshakeRequest(ByteString.Empty)).ConfigureAwait(false);

Check warning on line 291 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 291 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 291 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 291 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 291 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 291 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
await duplexStreamingCall.RequestStream.CompleteAsync().ConfigureAwait(false);

Check warning on line 292 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 292 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 292 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 292 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 292 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 292 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
var results = await duplexStreamingCall.ResponseStream.ToListAsync().ConfigureAwait(false);

Check warning on line 293 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 293 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 293 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 293 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 293 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.Single(results);
Assert.Equal("Done", results.First().Payload.ToStringUtf8());
{
var duplexStreamingCall = _flightClient.Handshake();
await duplexStreamingCall.RequestStream.WriteAsync(new FlightHandshakeRequest(ByteString.CopyFromUtf8("Hello"))).ConfigureAwait(false);

Check warning on line 304 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 304 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 304 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 304 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 304 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 304 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
await duplexStreamingCall.RequestStream.CompleteAsync().ConfigureAwait(false);

Check warning on line 305 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 305 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 305 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Ubuntu 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 305 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 305 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 305 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
var results = await duplexStreamingCall.ResponseStream.ToListAsync().ConfigureAwait(false);

Check warning on line 306 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 macOS 11 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)

Check warning on line 306 in csharp/test/Apache.Arrow.Flight.Tests/FlightTests.cs

GitHub Actions / AMD64 Windows 2019 18.04 C# 7.0.x

Test methods should not call ConfigureAwait(false), as it may bypass parallelization limits. Omit ConfigureAwait, or use ConfigureAwait(true) to avoid CA2007. (https://xunit.net/xunit.analyzers/rules/xUnit1030)
Assert.Single(results);
Assert.Equal("Hello handshake", results.First().Payload.ToStringUtf8());