Browse Source

Remove workaround for reactor-core issue

pull/23050/head
Rossen Stoyanchev 6 years ago
parent
commit
b3bc2d9253
  1. 4
      build.gradle
  2. 12
      spring-core/src/main/java/org/springframework/core/io/buffer/DataBufferUtils.java
  3. 5
      spring-core/src/test/java/org/springframework/core/io/buffer/DataBufferUtilsTests.java

4
build.gradle

@ -38,7 +38,7 @@ ext { @@ -38,7 +38,7 @@ ext {
kotlinVersion = "1.2.71"
log4jVersion = "2.11.2"
nettyVersion = "4.1.34.Final"
reactorVersion = "Californium-SR6"
reactorVersion = "Californium-BUILD-SNAPSHOT"
rxjavaVersion = "1.3.8"
rxjavaAdapterVersion = "1.2.1"
rxjava2Version = "2.2.8"
@ -149,6 +149,8 @@ configure(allprojects) { project -> @@ -149,6 +149,8 @@ configure(allprojects) { project ->
repositories {
maven { url "https://repo.spring.io/libs-release" }
maven { url "https://repo.spring.io/snapshot" } // Reactor
mavenLocal()
}
dependencies {

12
spring-core/src/main/java/org/springframework/core/io/buffer/DataBufferUtils.java

@ -57,12 +57,6 @@ public abstract class DataBufferUtils { @@ -57,12 +57,6 @@ public abstract class DataBufferUtils {
private static final Consumer<DataBuffer> RELEASE_CONSUMER = DataBufferUtils::release;
/**
* Workaround to disable use of pooled buffers:
* https://github.com/reactor/reactor-core/issues/1634.
*/
private static final DataBufferFactory defaultDataBufferFactory = new DefaultDataBufferFactory();
//---------------------------------------------------------------------
// Reading
@ -141,14 +135,12 @@ public abstract class DataBufferUtils { @@ -141,14 +135,12 @@ public abstract class DataBufferUtils {
Assert.isTrue(position >= 0, "'position' must be >= 0");
Assert.isTrue(bufferSize > 0, "'bufferSize' must be > 0");
DataBufferFactory bufferFactoryToUse = defaultDataBufferFactory;
Flux<DataBuffer> flux = Flux.using(channelSupplier,
channel -> Flux.create(sink -> {
ReadCompletionHandler handler =
new ReadCompletionHandler(channel, sink, position, bufferFactoryToUse, bufferSize);
new ReadCompletionHandler(channel, sink, position, bufferFactory, bufferSize);
sink.onDispose(handler::dispose);
DataBuffer dataBuffer = bufferFactoryToUse.allocateBuffer(bufferSize);
DataBuffer dataBuffer = bufferFactory.allocateBuffer(bufferSize);
ByteBuffer byteBuffer = dataBuffer.asByteBuffer(0, bufferSize);
channel.read(byteBuffer, position, dataBuffer, handler);
}),

5
spring-core/src/test/java/org/springframework/core/io/buffer/DataBufferUtilsTests.java

@ -35,7 +35,6 @@ import java.util.concurrent.CountDownLatch; @@ -35,7 +35,6 @@ import java.util.concurrent.CountDownLatch;
import io.netty.buffer.ByteBuf;
import org.junit.Before;
import org.junit.Ignore;
import org.junit.Test;
import org.mockito.stubbing.Answer;
import org.reactivestreams.Subscription;
@ -50,7 +49,9 @@ import org.springframework.core.io.Resource; @@ -50,7 +49,9 @@ import org.springframework.core.io.Resource;
import org.springframework.core.io.buffer.support.DataBufferTestUtils;
import static org.junit.Assert.*;
import static org.mockito.ArgumentMatchers.any;
import static org.mockito.ArgumentMatchers.*;
import static org.mockito.Mockito.anyLong;
import static org.mockito.Mockito.isA;
import static org.mockito.Mockito.*;
/**

Loading…
Cancel
Save