dannycjones commented on a change in pull request #3962:
URL: https://github.com/apache/hadoop/pull/3962#discussion_r815905811
##########
File path:
hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/S3ATestConstants.java
##########
@@ -97,6 +97,17 @@
*/
String DEFAULT_CSVTEST_FILE = LANDSAT_BUCKET + "scene_list.gz";
+ /**
+ * Configuration key for an existing object in a requester pays bucket:
{@value}.
+ * If not set, defaults to {@value DEFAULT_REQUESTER_PAYS_FILE}.
+ */
+ String KEY_REQUESTER_PAYS_FILE = TEST_FS_S3A + "requester-pays.file";
+
+ /**
+ * Default path for an S3 object inside a requester pays enabled bucket:
{@value}.
+ */
+ String DEFAULT_REQUESTER_PAYS_FILE =
"s3a://usgs-landsat/collection02/catalog.json";
Review comment:
I agree, though I'm less worried than with the original landsat
dependency as we're only relying on requester pays being enabled and an object
existing rather than specific file system structure or content.
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
