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

Upgrade C++ Arrow to 16.1.0 #462

Merged
merged 6 commits into from
Jun 21, 2024
Merged
Show file tree
Hide file tree
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
3 changes: 1 addition & 2 deletions .devcontainer/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,8 @@ FROM cpp AS vcpkg

USER vscode

# Copy our vcpkg manifest and ports
# Copy our vcpkg manifest
COPY --chown=vscode:vscode vcpkg.json /tmp/build/
COPY --chown=vscode:vscode ports /tmp/build/ports

# Install vcpkg
RUN sudo mkdir -p $VCPKG_ROOT \
Expand Down
3 changes: 1 addition & 2 deletions .dockerignore
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@
**
!vcpkg.json
!ports
!*/*.csproj
!*/*.csproj
2 changes: 0 additions & 2 deletions .github/workflows/devcontainer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,6 @@ on:
- "vcpkg.json"
- "*/*.csproj"
- ".github/workflows/devcontainer.yml"
- "ports/**"
pull_request:
branches: [master]
paths:
Expand All @@ -18,7 +17,6 @@ on:
- "vcpkg.json"
- "*/*.csproj"
- ".github/workflows/devcontainer.yml"
- "ports/**"
# Run once a week
schedule:
- cron: "34 2 * * 2"
Expand Down
4 changes: 3 additions & 1 deletion csharp.test/Encryption/TestEncryptionRoundTrip.cs
Original file line number Diff line number Diff line change
Expand Up @@ -177,7 +177,9 @@ public static void TestSingleWrapping()
// 1 for footer and 1 for each column, even though they use the same master key,
// as each data key needs to be encrypted separately by the master key rather than use a KEK.
Assert.That(testClient.WrappedKeys.Count, Is.EqualTo(3));
Assert.That(testClient.UnwrappedKeys.Count, Is.EqualTo(3));
// Column keys were previously cached and so would only be unwrapped once per column,
// but as of Arrow 16 this is no longer true and the unwrap count may be more.
Assert.That(testClient.UnwrappedKeys.Count, Is.GreaterThanOrEqualTo(3));
}

[Test]
Expand Down
4 changes: 2 additions & 2 deletions csharp.test/TestPhysicalTypeRoundtrip.cs
Original file line number Diff line number Diff line change
Expand Up @@ -163,7 +163,7 @@ private static void AssertReadRoundtrip(ResizableBuffer buffer, ExpectedColumn[]

var numRows = expectedColumns.First().Values.Length;

Assert.AreEqual("parquet-cpp-arrow version 15.0.2", fileMetaData.CreatedBy);
Assert.AreEqual("parquet-cpp-arrow version 16.1.0", fileMetaData.CreatedBy);
Assert.AreEqual(new Dictionary<string, string> {{"case", "Test"}, {"Awesome", "true"}}, fileMetaData.KeyValueMetadata);
Assert.AreEqual(expectedColumns.Length, fileMetaData.NumColumns);
Assert.AreEqual(numRows, fileMetaData.NumRows);
Expand All @@ -174,7 +174,7 @@ private static void AssertReadRoundtrip(ResizableBuffer buffer, ExpectedColumn[]
// The parquet format only stores an integer file version (1 or 2) and
// 2 gets mapped to the latest 2.x version.
Assert.AreEqual(ParquetVersion.PARQUET_2_6, fileMetaData.Version);
Assert.AreEqual("parquet-cpp-arrow version 15.0.2", fileMetaData.WriterVersion.ToString());
Assert.AreEqual("parquet-cpp-arrow version 16.1.0", fileMetaData.WriterVersion.ToString());

using var rowGroupReader = fileReader.RowGroup(0);
var rowGroupMetaData = rowGroupReader.MetaData;
Expand Down
2 changes: 1 addition & 1 deletion csharp.test/TestWriterProperties.cs
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ public static void TestDefaultProperties()
{
var p = WriterProperties.GetDefaultWriterProperties();

Assert.AreEqual("parquet-cpp-arrow version 15.0.2", p.CreatedBy);
Assert.AreEqual("parquet-cpp-arrow version 16.1.0", p.CreatedBy);
Assert.AreEqual(Compression.Uncompressed, p.Compression(new ColumnPath("anypath")));
Assert.AreEqual(int.MinValue, p.CompressionLevel(new ColumnPath("anypath")));
Assert.AreEqual(1024 * 1024, p.DataPageSize);
Expand Down
2 changes: 1 addition & 1 deletion csharp/ParquetSharp.csproj
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<GeneratePackageOnBuild>true</GeneratePackageOnBuild>
<TreatWarningsAsErrors>true</TreatWarningsAsErrors>
<NoWarn>1591;</NoWarn>
<VersionPrefix>15.0.2.1</VersionPrefix>
<VersionPrefix>16.1.0-beta1</VersionPrefix>
<Company>G-Research</Company>
<Authors>G-Research</Authors>
<Product>ParquetSharp</Product>
Expand Down
25 changes: 0 additions & 25 deletions ports/re2/portfile.cmake

This file was deleted.

13 changes: 0 additions & 13 deletions ports/re2/remove_include_dir.patch

This file was deleted.

18 changes: 0 additions & 18 deletions ports/re2/vcpkg.json

This file was deleted.

9 changes: 3 additions & 6 deletions vcpkg.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,17 +2,14 @@
"$schema": "https://raw.githubusercontent.com/microsoft/vcpkg-tool/main/docs/vcpkg.schema.json",
"name": "parquetsharp",
"version-string": "undefined",
"builtin-baseline": "eacb83982dd1d74950f36c3fbb6389d4a1c38982",
"builtin-baseline": "f7423ee180c4b7f40d43402c2feb3859161ef625",
"dependencies": [
"arrow"
],
"overrides": [
{
"name": "arrow",
"version": "15.0.2"
"version": "16.1.0#1"
}
],
"vcpkg-configuration": {
"overlay-ports": ["./ports"]
}
]
}
Loading