You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "snazy (via GitHub)" <gi...@apache.org> on 2025/01/15 08:09:26 UTC

[I] Tests should not "spam" system directories [polaris]

snazy opened a new issue, #758:
URL: https://github.com/apache/polaris/issues/758

   ### Describe the bug
   
   Some tests write test data to the system global `/tmp`:
   ```
   /tmp/buckets
   /tmp/ns
   /tmp/ns1
   /tmp/PolarisOverlappingTableTest
   /tmp/test-private*.pem
   /tmp/test-public*.pem
   ```
   
   This is bad practice and should be replaced with JUnit's `@TempDir`. It can also easily cause issues when tests are run repeatedly.
   
   ### To Reproduce
   
   _No response_
   
   ### Actual Behavior
   
   _No response_
   
   ### Expected Behavior
   
   _No response_
   
   ### Additional context
   
   _No response_
   
   ### System information
   
   _No response_


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Tests should not "spam" system directories [polaris]

Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on issue #758:
URL: https://github.com/apache/polaris/issues/758#issuecomment-2595914651

   This is a real issue causing false-positive test failures:
   ```
   Caused by: org.apache.hadoop.fs.FileAlreadyExistsException: File already exists: file:/tmp/ns1/my_table/metadata/v1.metadata.json
   	at org.apache.hadoop.fs.RawLocalFileSystem.create(RawLocalFileSystem.java:559)
   	at org.apache.hadoop.fs.RawLocalFileSystem.create(RawLocalFileSystem.java:597)
   	at org.apache.hadoop.fs.ChecksumFileSystem$ChecksumFSOutputSummer.<init>(ChecksumFileSystem.java:633)
   	at org.apache.hadoop.fs.ChecksumFileSystem.create(ChecksumFileSystem.java:721)
   	at org.apache.hadoop.fs.ChecksumFileSystem.create(ChecksumFileSystem.java:700)
   	at org.apache.hadoop.fs.FileSystem.create(FileSystem.java:1233)
   	at org.apache.hadoop.fs.FileSystem.create(FileSystem.java:1210)
   	at org.apache.hadoop.fs.FileSystem.create(FileSystem.java:1091)
   ```
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@polaris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org