KevinGG commented on a change in pull request #12452:
URL: https://github.com/apache/beam/pull/12452#discussion_r464721910



##########
File path: 
sdks/python/apache_beam/runners/interactive/interactive_runner_test.py
##########
@@ -66,6 +66,7 @@ class InteractiveRunnerTest(unittest.TestCase):
   def setUp(self):
     ie.new_env()
 
+  @unittest.skipIf(sys.platform == "win32", "[BEAM-10627]")

Review comment:
       Acked. It's a duplicate of BEAM-10514. The reason is the file system in 
test has a temp directory with a very long prefix, causing the file path 
exceeding windows default 260 character limit.
   
   A normal windows temp directory prefix: `c:\windows\temp\`, 
   In the test: `D:\a\beam\beam\sdks\python\target\.tox\py35-win\tmp\`.
   
   The length diff breaks the limit since usually Interactive Beam uses about 
240 characters in path length.




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to