apache / apache/beam

[Feature Request]: Support portable logical type for Beam SQL

Open
#36,682 6 comments 0 reactions 1 assignee Claimed by @Suvrat1629 View on GitHub
new feature P2 pinned
Dominant language
Java
Stars
8.7k
Forks
4.7k
Avg merge
1d 20h
Merged PRs (30d)
196

Description

### What would you like to happen?

Beam SQL only support certain java-only logical types hard-coded in place

https://github.com/apache/beam/blob/658b8f6daded95ff8490b8df9cb289dde6ba3294/sdks/java/extensions/sql/src/main/java/org/apache/beam/sdk/extensions/sql/impl/rel/BeamCalcRel.java#L604

which does not include standard portable logical type (portable dateime, decimal). As a result xlang and Beam YAML using Beam SQL does not support these types. We should add supports there. And for unknown logical type, we can use EncodedBytesLogicalType (#23374) once implemented.

### Issue Priority

Priority: 2 (default / most feature requests should be filed as P2)

### Issue Components

- [ ] Component: Python SDK
- [ ] Component: Java SDK
- [ ] Component: Go SDK
- [ ] Component: Typescript SDK
- [ ] Component: IO connector
- [ ] Component: Beam YAML
- [ ] Component: Beam examples
- [ ] Component: Beam playground
- [ ] Component: Beam katas
- [ ] Component: Website
- [ ] Component: Infrastructure
- [ ] Component: Spark Runner
- [ ] Component: Flink Runner
- [ ] Component: Samza Runner
- [ ] Component: Twister2 Runner
- [ ] Component: Hazelcast Jet Runner
- [ ] Component: Google Cloud Dataflow Runner

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.