This is inconsistent with the executor memory configuration. For executor memory, we have `spark.executor.memory` for heap size, and `spark.memory.offHeap.size` for off-heap size, and these 2 together is the total memory consumption for each executor process.    Now we have a single config for shuffle service total memory consumption, this seems better, shall we fix the executor memory config?