[
https://issues.apache.org/jira/browse/PHOENIX-7081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17812221#comment-17812221
]
ASF GitHub Bot commented on PHOENIX-7081:
-----------------------------------------
stoty commented on code in PR #1808:
URL: https://github.com/apache/phoenix/pull/1808#discussion_r1470804164
##########
phoenix-core/src/it/java/org/apache/phoenix/end2end/CreateTableIT.java:
##########
@@ -1557,7 +1558,7 @@ public void
testCoprocessorsForCreateIndexOnOldImplementation() throws Exception
// Now roll back to the old indexing
IndexUpgradeTool iut =
new IndexUpgradeTool(ROLLBACK_OP, tableName, null,
- "/tmp/index_upgrade_" +
UUID.randomUUID().toString(), false, null,
+ filePath + "/index_upgrade_" +
UUID.randomUUID().toString(), false, null,
Review Comment:
In the Hadoop Tools, path parameters are some times HDFS paths, so that is a
case where simple "/tmp" is the right solution.
Those are also separate HDFS file systems, so there is no danger of
coliision.
Have you checked that all of these instances are used as local paths, as
opposed to HadoopFS paths ?
(I have checked this one, and this is correct, as this is a local FS path)
##########
phoenix-core/src/it/java/org/apache/phoenix/end2end/CreateTableIT.java:
##########
@@ -89,6 +89,7 @@
@Category(ParallelStatsDisabledTest.class)
public class CreateTableIT extends ParallelStatsDisabledIT {
+ private static final String filePath =
System.getProperty("java.io.tmpdir");
Review Comment:
This name is not descriptive.
tmpPath would be better (in each file)
> Replace /tmp with {java.io.tmpdir} in tests
> -------------------------------------------
>
> Key: PHOENIX-7081
> URL: https://issues.apache.org/jira/browse/PHOENIX-7081
> Project: Phoenix
> Issue Type: Bug
> Components: core
> Reporter: Istvan Toth
> Assignee: Divneet Kaur
> Priority: Minor
> Labels: beginner, test
>
> I was running two test suites in parallel on a large VM, and it seems that
> OrphanViewToolIT cannot handle that.
> {noformat}
> java.io.FileNotFoundException: /tmp/OrphanView.txt (No such file or
> directory){noformat}
> Use some directory here that is under the source code's root.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
