apache/beam

Deprecate .withProjection from ParquetIO

Open

#20,770 opened on Jun 4, 2022

View on GitHub
 (1 comment) (0 reactions) (0 assignees)Java (4,097 forks)batch import
P3good first issueimprovementiojavaparquet

Repository metrics

Stars
 (7,313 stars)
PR merge metrics
 (Avg merge 6d) (243 merged PRs in 30d)

Description

There are multiple issues wrong with the API of withProjection:

  1. The current API requires an extra encoderSchema that is not needed when projecting data in Parquet. The simplest way to get this with the Parquet API is by passing the projectionSchema like this:

{color:#000000}AvroReadSupport{color}.setAvroReadSchema({color:#871094}conf{color}, {color:#871094}projectionSchema{color}); olor:#000000}AvroReadSupport{color}.setRequestedProjection({color:#871094}conf{color}, {color:#871094}projectionSchema{color}); We can offer an alternative method withProjection(Configuration conf, List<String\> fields) so users don't have to build their own projection Schema, but historically we have let users to rely on the upstream connector API. If we follow this we can better document in ParquetIO how to project fields by relying in the Parquet APIs and avoid maintaining this extra code in the Beam side.

 

Imported from Jira BEAM-11908. Original Jira may contain additional context. Reported by: iemejia.

Contributor guide