Skip to content

Commit d1a7207

Browse files
authored
Merge branch 'main' into dependabot/github_actions/actions/checkout-5
2 parents 92287d9 + 973e974 commit d1a7207

File tree

57 files changed

+1031
-253
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

57 files changed

+1031
-253
lines changed

.env

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@ ARCH_SHORT=amd64
4040

4141
# Default repository to pull and push images from
4242
REPO=ghcr.io/apache/arrow-java-dev
43-
ARROW_REPO=apache/arrow-dev
43+
ARROW_REPO=ghcr.io/apache/arrow-dev
4444

4545
# The setup attempts to generate coredumps by default, in order to disable the
4646
# coredump generation set it to 0
@@ -53,5 +53,4 @@ MAVEN=3.9.9
5353
# Versions for various dependencies used to build artifacts
5454
# Keep in sync with apache/arrow
5555
ARROW_REPO_ROOT=./arrow
56-
PYTHON=3.9
57-
VCPKG="f7423ee180c4b7f40d43402c2feb3859161ef625" # 2024.06.15 Release
56+
VCPKG="4334d8b4c8916018600212ab4dd4bbdc343065d1" # 2025.09.17 Release

.github/workflows/comment_bot.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ jobs:
3030
if: github.event.comment.body == 'take'
3131
runs-on: ubuntu-latest
3232
steps:
33-
- uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
33+
- uses: actions/github-script@v8
3434
with:
3535
github-token: ${{ secrets.GITHUB_TOKEN }}
3636
script: |-

.github/workflows/dev.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jobs:
3838
fetch-depth: 0
3939
persist-credentials: false
4040

41-
- uses: actions/setup-python@v5
41+
- uses: actions/setup-python@v6
4242
with:
4343
python-version: '3.x'
4444
- name: pre-commit (cache)

.github/workflows/dev_pr.yml

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ concurrency:
3535

3636
permissions:
3737
contents: read
38+
issues: write
3839
pull-requests: write
3940

4041
jobs:
@@ -49,28 +50,28 @@ jobs:
4950

5051
- name: Ensure PR title format
5152
id: title-format
52-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
53+
uses: actions/github-script@v8
5354
with:
5455
script: |
5556
const scripts = require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/dev_pr.js`);
5657
return scripts.check_title_format({core, github, context});
5758
5859
- name: Label PR
59-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
60+
uses: actions/github-script@v8
6061
with:
6162
script: |
6263
const scripts = require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/dev_pr.js`);
6364
await scripts.apply_labels({core, github, context});
6465
6566
- name: Ensure PR is labeled
66-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
67+
uses: actions/github-script@v8
6768
with:
6869
script: |
6970
const scripts = require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/dev_pr.js`);
7071
await scripts.check_labels({core, github, context});
7172
7273
- name: Ensure PR is linked to an issue
73-
uses: actions/github-script@60a0d83039c74a4aee543508d2ffcb1c3799cdea # v7.0.1
74+
uses: actions/github-script@v8
7475
with:
7576
script: |
7677
const scripts = require(`${process.env.GITHUB_WORKSPACE}/.github/workflows/dev_pr.js`);
@@ -80,9 +81,5 @@ jobs:
8081
if: '! github.event.pull_request.draft'
8182
env:
8283
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
83-
permissions:
84-
contents: read
85-
issues: write
86-
pull-requests: write
8784
run: |
8885
./.github/workflows/dev_pr_milestone.sh "${GITHUB_REPOSITORY}" ${{ github.event.number }}

.github/workflows/rc.yml

Lines changed: 29 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ jobs:
101101
packages: write
102102
steps:
103103
- name: Download source archive
104-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
104+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
105105
with:
106106
name: release-source
107107
- name: Extract source archive
@@ -133,7 +133,7 @@ jobs:
133133
with:
134134
repository: apache/parquet-testing
135135
path: arrow/cpp/submodules/parquet-testing
136-
- uses: docker/login-action@184bdaa0721073962dff0199f1fb9940f07167d1 # v3.5.0
136+
- uses: docker/login-action@5e57cd118135c172c3672efd75eb46360885c0ef # v3.6.0
137137
with:
138138
registry: ghcr.io
139139
username: ${{ github.actor }}
@@ -168,13 +168,13 @@ jobs:
168168
fail-fast: false
169169
matrix:
170170
platform:
171-
- { runs_on: macos-13, arch: "x86_64"}
171+
- { runs_on: macos-15-intel, arch: "x86_64"}
172172
- { runs_on: macos-14, arch: "aarch_64" }
173173
env:
174174
MACOSX_DEPLOYMENT_TARGET: "14.0"
175175
steps:
176176
- name: Download source archive
177-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
177+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
178178
with:
179179
name: release-source
180180
- name: Extract source archive
@@ -201,7 +201,7 @@ jobs:
201201
repository: apache/parquet-testing
202202
path: arrow/cpp/submodules/parquet-testing
203203
- name: Set up Python
204-
uses: actions/setup-python@0b93645e9fea7318ecaed2b359559ac225c90a2b # v5.3.0
204+
uses: actions/setup-python@v6
205205
with:
206206
cache: 'pip'
207207
python-version: 3.12
@@ -222,7 +222,7 @@ jobs:
222222
brew uninstall llvm || :
223223
224224
# We can remove this when we drop support for
225-
# macos-13. because macos-14 or later uses /opt/homebrew/
225+
# macos-15-intel. because macos-14 or later with arm64 uses /opt/homebrew/
226226
# not /usr/local/.
227227
#
228228
# Ensure updating python@XXX with the "--overwrite" option.
@@ -298,30 +298,36 @@ jobs:
298298
fail-fast: false
299299
matrix:
300300
platform:
301-
- runs_on: windows-2019
301+
- runs_on: windows-2022
302302
arch: "x86_64"
303303
steps:
304304
- name: Download source archive
305-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
305+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
306306
with:
307307
name: release-source
308308
- name: Extract source archive
309309
shell: bash
310310
run: |
311311
tar -xf apache-arrow-java-*.tar.gz --strip-components=1
312-
- name: Download the latest Apache Arrow C++
313-
if: github.event_name != 'schedule'
314-
shell: bash
315-
run: |
316-
ci/scripts/download_cpp.sh
312+
# We always use the main branch for apache/arrow for now.
313+
# Because we want to use
314+
# https://github.com/apache/arrow/pull/47749 in
315+
# apache/arrow-java. We can revert this workaround once Apache
316+
# Arrow 22.0.0 that includes the change released.
317+
#
318+
# - name: Download the latest Apache Arrow C++
319+
# if: github.event_name != 'schedule'
320+
# shell: bash
321+
# run: |
322+
# ci/scripts/download_cpp.sh
317323
- name: Checkout Apache Arrow C++
318-
if: github.event_name == 'schedule'
324+
# if: github.event_name == 'schedule'
319325
uses: actions/checkout@v5
320326
with:
321327
repository: apache/arrow
322328
path: arrow
323329
- name: Set up Java
324-
uses: actions/setup-java@7a6d8a8234af8eb26422e24e3006232cccaa061b # v4.6.0
330+
uses: actions/setup-java@v5
325331
with:
326332
java-version: '11'
327333
distribution: 'temurin'
@@ -354,7 +360,7 @@ jobs:
354360
- name: Build
355361
shell: cmd
356362
run: |
357-
call "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Auxiliary\Build\vcvarsall.bat" x64
363+
call "C:\Program Files\Microsoft Visual Studio\2022\Enterprise\VC\Auxiliary\Build\vcvarsall.bat" x64
358364
REM For ORC
359365
set TZDIR=/c/msys64/usr/share/zoneinfo
360366
bash -c "ci/scripts/jni_windows_build.sh . arrow build jni"
@@ -375,7 +381,7 @@ jobs:
375381
- jni-windows
376382
steps:
377383
- name: Download artifacts
378-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
384+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
379385
with:
380386
path: artifacts
381387
- name: Decompress artifacts
@@ -452,15 +458,15 @@ jobs:
452458
contents: read
453459
packages: write
454460
steps:
455-
- uses: actions/setup-python@0b93645e9fea7318ecaed2b359559ac225c90a2b # v5.3.0
461+
- uses: actions/setup-python@v6
456462
with:
457463
cache: 'pip'
458464
- name: Download source archive
459-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
465+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
460466
with:
461467
name: release-source
462468
- name: Download Javadocs
463-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
469+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
464470
with:
465471
name: reference
466472
- name: Extract source archive
@@ -525,7 +531,7 @@ jobs:
525531
cp ../.asf.yaml ./
526532
git add .nojekyll .asf.yaml
527533
- name: Download
528-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
534+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
529535
with:
530536
name: release-html
531537
- name: Extract
@@ -561,7 +567,7 @@ jobs:
561567
- ubuntu-latest
562568
steps:
563569
- name: Download release artifacts
564-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
570+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
565571
with:
566572
pattern: release-*
567573
- name: Verify
@@ -595,7 +601,7 @@ jobs:
595601
contents: write
596602
steps:
597603
- name: Download release artifacts
598-
uses: actions/download-artifact@634f93cb2916e3fdff6788551b99b062d0335ce0 # v5.0.0
604+
uses: actions/download-artifact@018cc2cf5baa6db3ef3c5f8a56943fffe632ef53 # v6.0.0
599605
with:
600606
pattern: release-*
601607
path: artifacts

.github/workflows/test.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -196,7 +196,7 @@ jobs:
196196
key: integration-conda-${{ hashFiles('cpp/**') }}
197197
restore-keys: integration-conda-
198198
- name: Setup Python
199-
uses: actions/setup-python@v5
199+
uses: actions/setup-python@v6
200200
with:
201201
python-version: 3.12
202202
- name: Setup Archery

adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/CompositeJdbcConsumer.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@
2424
import org.apache.arrow.util.AutoCloseables;
2525
import org.apache.arrow.vector.ValueVector;
2626
import org.apache.arrow.vector.VectorSchemaRoot;
27-
import org.apache.arrow.vector.types.pojo.ArrowType;
2827

2928
/** Composite consumer which hold all consumers. It manages the consume and cleanup process. */
3029
public class CompositeJdbcConsumer implements JdbcConsumer {
@@ -46,9 +45,9 @@ public void consume(ResultSet rs) throws SQLException, IOException {
4645
BaseConsumer consumer = (BaseConsumer) consumers[i];
4746
JdbcFieldInfo fieldInfo =
4847
new JdbcFieldInfo(rs.getMetaData(), consumer.columnIndexInResultSet);
49-
ArrowType arrowType = consumer.vector.getMinorType().getType();
48+
5049
throw new JdbcConsumerException(
51-
"Exception while consuming JDBC value", e, fieldInfo, arrowType);
50+
"Exception while consuming JDBC value", e, fieldInfo, consumer.vector.getField());
5251
} else {
5352
throw e;
5453
}

adapter/jdbc/src/main/java/org/apache/arrow/adapter/jdbc/consumer/exceptions/JdbcConsumerException.java

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -17,33 +17,33 @@
1717
package org.apache.arrow.adapter.jdbc.consumer.exceptions;
1818

1919
import org.apache.arrow.adapter.jdbc.JdbcFieldInfo;
20-
import org.apache.arrow.vector.types.pojo.ArrowType;
20+
import org.apache.arrow.vector.types.pojo.Field;
2121

2222
/**
2323
* Exception while consuming JDBC data. This exception stores the JdbcFieldInfo for the column and
2424
* the ArrowType for the corresponding vector for easier debugging.
2525
*/
2626
public class JdbcConsumerException extends RuntimeException {
2727
final JdbcFieldInfo fieldInfo;
28-
final ArrowType arrowType;
28+
final Field field;
2929

3030
/**
3131
* Construct JdbcConsumerException with all fields.
3232
*
3333
* @param message error message
3434
* @param cause original exception
3535
* @param fieldInfo JdbcFieldInfo for the column
36-
* @param arrowType ArrowType for the corresponding vector
36+
* @param field ArrowType for the corresponding vector
3737
*/
3838
public JdbcConsumerException(
39-
String message, Throwable cause, JdbcFieldInfo fieldInfo, ArrowType arrowType) {
39+
String message, Throwable cause, JdbcFieldInfo fieldInfo, Field field) {
4040
super(message, cause);
4141
this.fieldInfo = fieldInfo;
42-
this.arrowType = arrowType;
42+
this.field = field;
4343
}
4444

45-
public ArrowType getArrowType() {
46-
return this.arrowType;
45+
public Field getField() {
46+
return this.field;
4747
}
4848

4949
public JdbcFieldInfo getFieldInfo() {

bom/pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -165,7 +165,7 @@ under the License.
165165
<version>${project.version}</version>
166166
</dependency>
167167
<dependency>
168-
<groupId>org.apache.arrow</groupId>
168+
<groupId>org.apache.arrow.gandiva</groupId>
169169
<artifactId>arrow-gandiva</artifactId>
170170
<version>${project.version}</version>
171171
</dependency>

c/src/main/cpp/jni_wrapper.cc

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -327,19 +327,20 @@ void ArrowArrayStreamRelease(ArrowArrayStream* stream) {
327327

328328
jint JNI_OnLoad(JavaVM* vm, void* reserved) {
329329
JNIEnv* env;
330-
if (vm->GetEnv(reinterpret_cast<void**>(&env), JNI_VERSION) != JNI_OK) {
331-
return JNI_ERR;
330+
const int err_code = vm->GetEnv(reinterpret_cast<void**>(&env), JNI_VERSION);
331+
if (err_code != JNI_OK) {
332+
return err_code;
332333
}
333334
JNI_METHOD_START
334-
kObjectClass = CreateGlobalClassReference(env, "Ljava/lang/Object;");
335+
kObjectClass = CreateGlobalClassReference(env, "java/lang/Object");
335336
kRuntimeExceptionClass =
336-
CreateGlobalClassReference(env, "Ljava/lang/RuntimeException;");
337+
CreateGlobalClassReference(env, "java/lang/RuntimeException");
337338
kPrivateDataClass =
338-
CreateGlobalClassReference(env, "Lorg/apache/arrow/c/jni/PrivateData;");
339+
CreateGlobalClassReference(env, "org/apache/arrow/c/jni/PrivateData");
339340
kCDataExceptionClass =
340-
CreateGlobalClassReference(env, "Lorg/apache/arrow/c/jni/CDataJniException;");
341+
CreateGlobalClassReference(env, "org/apache/arrow/c/jni/CDataJniException");
341342
kStreamPrivateDataClass = CreateGlobalClassReference(
342-
env, "Lorg/apache/arrow/c/ArrayStreamExporter$ExportedArrayStreamPrivateData;");
343+
env, "org/apache/arrow/c/ArrayStreamExporter$ExportedArrayStreamPrivateData");
343344

344345
kPrivateDataLastErrorField =
345346
GetFieldID(env, kStreamPrivateDataClass, "lastError", "[B");

0 commit comments

Comments
 (0)