tengqm commented on code in PR #49:
URL: https://github.com/apache/gravitino-site/pull/49#discussion_r1987357612


##########
docs/getting-started/aws-remote-access.md:
##########
@@ -0,0 +1,36 @@
+---
+title: "Remote Access for Apache Gravitino on AWS"
+slug: /getting-started/aws-remote-access
+license: "This software is licensed under the Apache License version 2."
+---
+
+## Accessing Apache Gravitino on AWS externally
+
+When you deploy Gravitino on AWS, accessing it externally requires
+some additional configuration due to how AWS networking works.

Review Comment:
   > A grammar checker can see and visually display the issues right away in 
the GitHub interface or in an IDE.
   
   Can you tell me where is the guideline on using grammar checkers?
   I'm always using vim (as my IDE). Is this line-wrapping thing related to IDE?
   
   > To see any problems like this, a reviewer must manually remove the new 
lines.
   
   Give me an example please? Why we have to remove this new lines before 
review?
   Why we cannot just treat them as a book?
   
   > This means it's more work to review these changes, and as you can see, it 
introduces errors that require more work to fix.
   
   If we have to remove new lines to review, then I tend to agree to remove 
these new lines.
   
   
   



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

Reply via email to