Skip to content

Commit

Permalink
Upgrade C++ Arrow to 16.1.0 (#462)
Browse files Browse the repository at this point in the history
  • Loading branch information
adamreeve authored Jun 21, 2024
1 parent 9e3cfa5 commit f928ac7
Show file tree
Hide file tree
Showing 11 changed files with 12 additions and 73 deletions.
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"]
}
]
}

0 comments on commit f928ac7

Please sign in to comment.