ayushtkn commented on code in PR #5475:
URL: https://github.com/apache/hadoop/pull/5475#discussion_r1171522531
##########
hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/troubleshooting_s3a.md:
##########
@@ -1179,7 +1179,8 @@ file using configured SSE-C keyB into that structure.
### Instruction file not found for S3 object
Reading an unencrypted file would fail when read through CSE enabled client.
-```
+
+```txt
Review Comment:
Why?
##########
hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/troubleshooting_s3a.md:
##########
@@ -1216,15 +1217,16 @@ java.lang.SecurityException: Instruction file not found
for S3 object with bucke
at org.apache.hadoop.util.ToolRunner.run(ToolRunner.java:95)
at org.apache.hadoop.fs.FsShell.main(FsShell.java:390)
```
+
Review Comment:
unrelated?
##########
hadoop-tools/hadoop-aws/src/site/markdown/tools/hadoop-aws/troubleshooting_s3a.md:
##########
@@ -1216,15 +1217,16 @@ java.lang.SecurityException: Instruction file not found
for S3 object with bucke
at org.apache.hadoop.util.ToolRunner.run(ToolRunner.java:95)
at org.apache.hadoop.fs.FsShell.main(FsShell.java:390)
```
+
CSE enabled client should read encrypted data only.
### CSE-KMS method requires KMS key ID
KMS key ID is required for CSE-KMS to encrypt data, not providing one leads
to failure.
-```
-2021-07-07 11:33:04,550 WARN fs.FileSystem: Failed to initialize fileystem
+```txt
+2021-07-07 11:33:04,550 WARN fs.FileSystem: Failed to initialize filesystem
Review Comment:
just fix typo
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
