Optional.stream()

[osmanpasha]

Я бы предположил, что по смыслу при null-аргументе лучше всё-таки кидать IllegalArgumentException, т.к. вызвать этот метод, не подав в него нормальный идентификатор - это таки ошибка.

[alexdoublesmile]

а почему нельзя просто аргумент метода проверять с помощью javax.validation?

public BigDecimal getOrderPrice(@NotNull Long orderId) {
        return orderRepository.findByOrderId(orderId)
                .stream()
                .map(OrderLine::getPrice)
                .reduce(BigDecimal.ZERO, BigDecimal::add);
}

И никакие optional тут не нужны. Читаемый рабочий код