chamikaramj commented on a change in pull request #15416:
URL: https://github.com/apache/beam/pull/15416#discussion_r718788980
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/io/FileBasedSink.java
##########
@@ -522,16 +526,17 @@ protected static ResourceId
buildTemporaryFilename(ResourceId tempDirectory, Str
* Constructs a WriteOperation using the default strategy for generating a
temporary directory
* from the base output filename.
*
- * <p>Default is a uniquely named subdirectory of the provided
tempDirectory, e.g. if
- * tempDirectory is /path/to/foo/, the temporary directory will be
+ * <p>Without windowing, the default is a uniquely named subdirectory of
the provided
+ * tempDirectory, e.g. if tempDirectory is /path/to/foo/, the temporary
directory will be
* /path/to/foo/.temp-beam-$uuid.
*
+ * <p>With windowing, the default is to use tempDirectory directly.
Review comment:
Can we explain why we do this ?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
