Made the BasicCompletableFuture inner class of the ExecutionServiceImpl

Since this class is not meant for public usage since it is in the SPI.impl package. I made it an internal class of the ExecutorServiceImpl since that class is the only user. Once other classes need it, we can always pull it out.