Skip to content

Conversation

@RustedBones
Copy link
Contributor

See #33412

Copy link
Contributor

@Abacn Abacn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick fix!

will merge after tests passed

@github-actions
Copy link
Contributor

Assigning reviewers. If you would like to opt out of this review, comment assign to next reviewer:

R: @robertwb for label java.
R: @shunping for label io.

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

@Abacn
Copy link
Contributor

Abacn commented Jan 22, 2025

confirmed it fixed issue by running ./gradlew :it:google-cloud-platform:BigQueryPerformanceTest -info

before:

org.apache.beam.it.gcp.bigquery.BigQueryIOLT > testStorageAPIWriteThenRead FAILED
    org.apache.beam.sdk.Pipeline$PipelineExecutionException: java.lang.ClassCastException: class [B cannot be cast to class java.nio.ByteBuffer ([B and java.nio.ByteBuffer are in module java.base of loader 'bootstrap')
        Caused by:
        java.lang.ClassCastException: class [B cannot be cast to class java.nio.ByteBuffer ([B and java.nio.ByteBuffer are in module java.base of loader 'bootstrap')
            at org.apache.beam.sdk.io.gcp.bigquery.AvroGenericRecordToStorageApiProto.lambda$static$4(AvroGenericRecordToStorageApiProto.java:89)

after

test passed

@Abacn Abacn merged commit 7313b9a into apache:master Jan 22, 2025
15 checks passed
VardhanThigle pushed a commit to VardhanThigle/beam that referenced this pull request Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants