flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Chesnay Schepler (JIRA)" <j...@apache.org>
Subject [jira] [Closed] (FLINK-9976) Odd signatures for streaming file sink format builders
Date Tue, 31 Jul 2018 09:24:01 GMT

     [ https://issues.apache.org/jira/browse/FLINK-9976?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Chesnay Schepler closed FLINK-9976.
-----------------------------------
    Resolution: Fixed

master: 23161ee91391fae4734a6f2f56e95e00d5d298ec
1.6: a3bd72d39ddec9681b5296bc52c9eba415050220

> Odd signatures for streaming file sink format builders
> ------------------------------------------------------
>
>                 Key: FLINK-9976
>                 URL: https://issues.apache.org/jira/browse/FLINK-9976
>             Project: Flink
>          Issue Type: Bug
>          Components: Streaming Connectors
>    Affects Versions: 1.6.0
>            Reporter: Chesnay Schepler
>            Assignee: Chesnay Schepler
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.6.0
>
>
> There are 2 instances of apparently unnecessary generic parameters in the format builders
for the {{StreamingFileSink}}.
> Both these methods have a generic parameter for the BucketID type, however the builder
itself already has such a parameter. The methods use unchecked casts to make the types fit,
so we should be able to modify the signature to use the builders parameter instead.
> {code}
> public static class RowFormatBuilder<IN, BucketID> extends StreamingFileSink.BucketsBuilder<IN,
BucketID> {
> ...
> 	public <ID> StreamingFileSink.RowFormatBuilder<IN, ID> withBucketerAndPolicy(final
Bucketer<IN, ID> bucketer, final RollingPolicy<IN, ID> policy) {
> 		@SuppressWarnings("unchecked")
> 		StreamingFileSink.RowFormatBuilder<IN, ID> reInterpreted = (StreamingFileSink.RowFormatBuilder<IN,
ID>) this;
> 		reInterpreted.bucketer = Preconditions.checkNotNull(bucketer);
> 		reInterpreted.rollingPolicy = Preconditions.checkNotNull(policy);
> 		return reInterpreted;
> 	}
> ...
> {code}
> {code}
> public static class BulkFormatBuilder<IN, BucketID> extends StreamingFileSink.BucketsBuilder<IN,
BucketID> {
> ...
> 	public <ID> StreamingFileSink.BulkFormatBuilder<IN, ID> withBucketer(Bucketer<IN,
ID> bucketer) {
> 		@SuppressWarnings("unchecked")
> 		StreamingFileSink.BulkFormatBuilder<IN, ID> reInterpreted = (StreamingFileSink.BulkFormatBuilder<IN,
ID>) this;
> 		reInterpreted.bucketer = Preconditions.checkNotNull(bucketer);
> 		return reInterpreted;
> 	}
> ...
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Mime
View raw message