kaygorodov commented on code in PR #25667:
URL: https://github.com/apache/flink/pull/25667#discussion_r1849982828


##########
docs/content/docs/dev/datastream/sources.md:
##########
@@ -325,16 +325,13 @@ public class FixedFetcherSizeSourceReader<E, T, SplitT 
extends SourceSplit, Spli
         extends SourceReaderBase<E, T, SplitT, SplitStateT> {
 
     public FixedFetcherSizeSourceReader(
-            FutureCompletingBlockingQueue<RecordsWithSplitIds<E>> 
elementsQueue,
             Supplier<SplitReader<E, SplitT>> splitFetcherSupplier,
             RecordEmitter<E, T, SplitStateT> recordEmitter,
             Configuration config,
             SourceReaderContext context) {
         super(
-                elementsQueue,
                 new FixedSizeSplitFetcherManager<>(
-                        config.getInteger(SourceConfig.NUM_FETCHERS),
-                        elementsQueue,
+                        config.get(SourceConfig.NUM_FETCHERS),
                         splitFetcherSupplier),

Review Comment:
   Maybe, I am missing something. It was actually passed :
   ```java
                   new FixedSizeSplitFetcherManager<>(
                           config.getInteger(SourceConfig.NUM_FETCHERS),
   ```
   what I did is changed the removed _getInteger_ to _get_. This code compiles 
now.
   
   FixedSizeSplitFetcherManager expects an int as the first parameter in its 
constructor:
   
   ```java
       public FixedSizeSplitFetcherManager(
               int numFetchers,
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to