我正在研究一个库,它将采用一个对象DataRequest
作为输入参数并基于该对象构建一个URL,然后调用我们的应用程序服务器使用apache http客户端,然后将响应返回给使用我们库的客户。有些客户会拨打executeSync
方法获得相同的功能,一些客户会拨打我们的executeAsync
方法来获取数据。在多线程环境中并行执行每个子任务
executeSync()
- 等待,直到我有一个结果,返回结果。executeAsync()
- 返回一个Future,如果需要,可以在其他事情完成后立即处理。
下面是我DataClient
类具有以上两种方法:
public class DataClient implements Client {
private final ForkJoinPool forkJoinPool = new ForkJoinPool(16);
private CloseableHttpClient httpClientBuilder;
// initializing httpclient only once
public DataClient() {
try {
RequestConfig requestConfig =
RequestConfig.custom().setConnectionRequestTimeout(500).setConnectTimeout(500)
.setSocketTimeout(500).setStaleConnectionCheckEnabled(false).build();
SocketConfig socketConfig =
SocketConfig.custom().setSoKeepAlive(true).setTcpNoDelay(true).build();
PoolingHttpClientConnectionManager poolingHttpClientConnectionManager =
new PoolingHttpClientConnectionManager();
poolingHttpClientConnectionManager.setMaxTotal(300);
poolingHttpClientConnectionManager.setDefaultMaxPerRoute(200);
httpClientBuilder =
HttpClientBuilder.create().setConnectionManager(poolingHttpClientConnectionManager)
.setDefaultRequestConfig(requestConfig).setDefaultSocketConfig(socketConfig).build();
} catch (Exception ex) {
// log error
}
}
@Override
public List<DataResponse> executeSync(DataRequest key) {
List<DataResponse> responsList = null;
Future<List<DataResponse>> responseFuture = null;
try {
responseFuture = executeAsync(key);
responsList = responseFuture.get(key.getTimeout(), key.getTimeoutUnit());
} catch (TimeoutException | ExecutionException | InterruptedException ex) {
responsList =
Collections.singletonList(new DataResponse(DataErrorEnum.CLIENT_TIMEOUT,
DataStatusEnum.ERROR));
responseFuture.cancel(true);
// logging exception here
}
return responsList;
}
@Override
public Future<List<DataResponse>> executeAsync(DataRequest key) {
DataFetcherTask task = new DataFetcherTask(key, this.httpClientBuilder);
return this.forkJoinPool.submit(task);
}
}
下面是我DataFetcherTask
类也有一个静态类DataRequestTask
它通过使URL调用我们的应用服务器:
public class DataFetcherTask extends RecursiveTask<List<DataResponse>> {
private final DataRequest key;
private final CloseableHttpClient httpClientBuilder;
public DataFetcherTask(DataRequest key, CloseableHttpClient httpClientBuilder) {
this.key = key;
this.httpClientBuilder = httpClientBuilder;
}
@Override
protected List<DataResponse> compute() {
// Create subtasks for the key and invoke them
List<DataRequestTask> requestTasks = requestTasks(generateKeys());
invokeAll(requestTasks);
// All tasks are finished if invokeAll() returns.
List<DataResponse> responseList = new ArrayList<>(requestTasks.size());
for (DataRequestTask task : requestTasks) {
try {
responseList.add(task.get());
} catch (InterruptedException | ExecutionException e) {
Thread.currentThread().interrupt();
return Collections.emptyList();
}
}
return responseList;
}
private List<DataRequestTask> requestTasks(List<DataRequest> keys) {
List<DataRequestTask> tasks = new ArrayList<>(keys.size());
for (DataRequest key : keys) {
tasks.add(new DataRequestTask(key));
}
return tasks;
}
// In this method I am making a HTTP call to another service
// and then I will make List<DataRequest> accordingly.
private List<DataRequest> generateKeys() {
List<DataRequest> keys = new ArrayList<>();
// use key object which is passed in contructor to make HTTP call to another service
// and then make List of DataRequest object and return keys.
return keys;
}
/** Inner class for the subtasks. */
private static class DataRequestTask extends RecursiveTask<DataResponse> {
private final DataRequest request;
public DataRequestTask(DataRequest request) {
this.request = request;
}
@Override
protected DataResponse compute() {
return performDataRequest(this.request);
}
private DataResponse performDataRequest(DataRequest key) {
MappingHolder mappings = DataMapping.getMappings(key.getType());
List<String> hostnames = mappings.getAllHostnames(key);
for (String hostname : hostnames) {
String url = generateUrl(hostname);
HttpGet httpGet = new HttpGet(url);
httpGet.setConfig(generateRequestConfig());
httpGet.addHeader(key.getHeader());
try (CloseableHttpResponse response = httpClientBuilder.execute(httpGet)) {
HttpEntity entity = response.getEntity();
String responseBody =
TestUtils.isEmpty(entity) ? null : IOUtils.toString(entity.getContent(),
StandardCharsets.UTF_8);
return new DataResponse(responseBody, DataErrorEnum.OK, DataStatusEnum.OK);
} catch (IOException ex) {
// log error
}
}
return new DataResponse(DataErrorEnum.SERVERS_DOWN, DataStatusEnum.ERROR);
}
}
}
对于每个DataRequest
对象,都有一个DataResponse
对象。现在有人通过传递DataRequest
对象调用我们的库,在内部我们制作List<DataRequest>
对象,然后我们并行调用每个对象DataRequest
,并返回List<DataResponse>
,其中列表中的每个DataResponse
对象都会响应对应的对象DataRequest
对象。
下面是流量:
- 客户将通过传递
DataRequest
对象调用DataClient
类。他们可以根据他们的要求调用executeSync()
或executeAsync()
方法。 - 现在在
DataFetcherTask
类(这是ForkJoinTask's
亚型一个RecursiveTask
一个),给定key
对象,它是一个单一的DataRequest
,我会产生List<DataRequest>
,然后调用每个子任务并行为列表中的每个对象DataRequest
。这些子任务与父任务在相同的ForkJoinPool
中执行。 - 现在在
DataRequestTask
类中,我通过创建URL并将其DataResponse
对象返回来执行每个DataRequest
对象。
问题陈述:
因为这个库被称为一个非常高的吞吐量的环境,因此必须非常快。对于同步调用,在单独的线程中执行可以吗?这将导致线程的额外成本和资源以及线程的上下文切换成本,所以我有点混淆。此外,我在这里使用ForkJoinPool
这将节省我使用额外的线程池,但它是在这里正确的选择?
有没有更好的和有效的方式来做同样的事情,也可以提高性能?我使用的是Java 7,并且可以访问Guava库,所以如果它可以简化任何事情,那么我也可以开放它。
看起来我们在负载很重时会看到一些争用。有没有什么办法可以使这个代码在非常重的负载下运行时进入线程争用?
听起来像[ThreadPool](https://docs.oracle.com/javase/7/docs/api/java/util/concurrent/ThreadPoolExecutor.html)会很有用,但请记住,过早优化是所有邪恶的来源 –
@ScaryWombat同意,这就是为什么我会做负载测试,但问题是我使用ForkJoinPool这也是ThreadPool的专用形式是合理的。然后,我使用executeSync方法的方式是否正确? – john
你看到了什么样的争用?也许是重负载新的ForkJoinPool(16);'是不够的,尝试增加'16'到一个更大的值 – Teg