Skip to content

Commit fe2c7cf

Browse files
committed
nits: Minor refactoring to remove some unused class members.
1 parent 46def32 commit fe2c7cf

File tree

1 file changed

+2
-10
lines changed

1 file changed

+2
-10
lines changed

xds/src/main/java/io/grpc/xds/ExternalProcessorFilter.java

Lines changed: 2 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ public ConfigOrError<? extends FilterConfig> parseFilterConfigOverride(Message r
102102
@Override
103103
public ClientInterceptor buildClientInterceptor(FilterConfig filterConfig,
104104
@Nullable FilterConfig overrideConfig, ScheduledExecutorService scheduler) {
105-
return new ExternalProcessorInterceptor(this, (ExternalProcessorFilterConfig) filterConfig, overrideConfig, scheduler);
105+
return new ExternalProcessorInterceptor((ExternalProcessorFilterConfig) filterConfig);
106106
}
107107

108108
static final class ExternalProcessorFilterConfig implements FilterConfig {
@@ -123,10 +123,7 @@ public String typeUrl() {
123123

124124
static final class ExternalProcessorInterceptor implements ClientInterceptor {
125125
private final CachedChannelManager cachedChannelManager = new CachedChannelManager();
126-
private final ExternalProcessorFilter filter;
127126
private final ExternalProcessorFilterConfig filterConfig;
128-
private final FilterConfig overrideConfig;
129-
private final ScheduledExecutorService scheduler;
130127

131128
private static final MethodDescriptor.Marshaller<InputStream> RAW_MARSHALLER =
132129
new MethodDescriptor.Marshaller<InputStream>() {
@@ -136,13 +133,8 @@ static final class ExternalProcessorInterceptor implements ClientInterceptor {
136133
public InputStream parse(InputStream stream) { return stream; }
137134
};
138135

139-
ExternalProcessorInterceptor(ExternalProcessorFilter filter,
140-
ExternalProcessorFilterConfig filterConfig,
141-
@Nullable FilterConfig overrideConfig, ScheduledExecutorService scheduler) {
142-
this.filter = filter;
136+
ExternalProcessorInterceptor(ExternalProcessorFilterConfig filterConfig) {
143137
this.filterConfig = filterConfig;
144-
this.overrideConfig = overrideConfig;
145-
this.scheduler = scheduler;
146138
}
147139

148140
@Override

0 commit comments

Comments
 (0)