You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tez.apache.org by je...@apache.org on 2020/02/27 15:06:04 UTC
[tez] branch branch-0.9 updated: TEZ-4127.
TestMergeManager#testOnDiskMergerFilenames fails with dot directory name
This is an automated email from the ASF dual-hosted git repository.
jeagles pushed a commit to branch branch-0.9
in repository https://gitbox.apache.org/repos/asf/tez.git
The following commit(s) were added to refs/heads/branch-0.9 by this push:
new e4a08a2 TEZ-4127. TestMergeManager#testOnDiskMergerFilenames fails with dot directory name
e4a08a2 is described below
commit e4a08a26fb23eafa6ee13a727d8c4d70a5acbab5
Author: Jonathan Eagles <je...@apache.org>
AuthorDate: Thu Feb 27 09:04:15 2020 -0600
TEZ-4127. TestMergeManager#testOnDiskMergerFilenames fails with dot directory name
(cherry picked from commit f12fd5caf965680aa0c56adbe963dad3dca8bd3f)
---
.../library/common/shuffle/orderedgrouped/TestMergeManager.java | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/orderedgrouped/TestMergeManager.java b/tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/orderedgrouped/TestMergeManager.java
index 5737578..92d9125 100644
--- a/tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/orderedgrouped/TestMergeManager.java
+++ b/tez-runtime-library/src/test/java/org/apache/tez/runtime/library/common/shuffle/orderedgrouped/TestMergeManager.java
@@ -757,9 +757,9 @@ public class TestMergeManager {
assertEquals(m2Path.toString().length(), m3Path.toString().length());
// Ensure the filenames are used correctly - based on the first file given to the merger.
- String m1Prefix = m1Path.toString().substring(0, m1Path.toString().indexOf("."));
- String m2Prefix = m2Path.toString().substring(0, m2Path.toString().indexOf("."));
- String m3Prefix = m3Path.toString().substring(0, m3Path.toString().indexOf("."));
+ String m1Prefix = m1Path.toString().substring(0, m1Path.toString().lastIndexOf('.'));
+ String m2Prefix = m2Path.toString().substring(0, m2Path.toString().lastIndexOf('.'));
+ String m3Prefix = m3Path.toString().substring(0, m3Path.toString().lastIndexOf('.'));
assertEquals(m1Prefix, m2Prefix);
assertNotEquals(m1Prefix, m3Prefix);