[CELEBORN-1478] Fix wrong use partitionId as shuffleId when readParti…

…tion

### What changes were proposed in this pull request?
Change partitionId to shuffleId

### Why are the changes needed?
As title, Although the partitionId is used as the shuffleId when reading a partition, it has no effect on the final result

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
PASS GA

Closes apache#2589 from RexXiong/CELEBORN-1478.

Authored-by: Shuang <[email protected]>
Signed-off-by: SteNicholas <[email protected]>

client/src/main/java/org/apache/celeborn/client/ShuffleClientImpl.java

@@ -1679,7 +1679,7 @@ public CelebornInputStream readPartition(
       int[] mapAttempts,
       MetricsCallback metricsCallback)
       throws IOException {
-    if (partitionId == Utils$.MODULE$.UNKNOWN_APP_SHUFFLE_ID()) {
+    if (shuffleId == Utils$.MODULE$.UNKNOWN_APP_SHUFFLE_ID()) {
       logger.warn("Shuffle data is empty for shuffle {}: UNKNOWN_APP_SHUFFLE_ID.", shuffleId);
       return CelebornInputStream.empty();
     }