Jason Fehr has posted comments on this change. ( http://gerrit.cloudera.org:8080/21929 )
Change subject: Add blog post for Impala talks at Community Over Code NA 2024 ...................................................................... Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/21929/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/21929/2//COMMIT_MSG@7 PS2, Line 7: Add blog post for Impala talks at Community Over Code NA 2024 Can we prefix these commits with wording that makes it easy to identify blog related changes (similar to how we prefix code changes with the Jira id)? Possible ideas are "BLOG: " or "SITE: " http://gerrit.cloudera.org:8080/#/c/21929/2/nikola_site_generator/posts/healing-iceberg-tables-with-impala.md File nikola_site_generator/posts/healing-iceberg-tables-with-impala.md: http://gerrit.cloudera.org:8080/#/c/21929/2/nikola_site_generator/posts/healing-iceberg-tables-with-impala.md@2 PS2, Line 2: t The 'T' should be capitalized. http://gerrit.cloudera.org:8080/#/c/21929/2/nikola_site_generator/posts/impalas-living-on-iceberg.md File nikola_site_generator/posts/impalas-living-on-iceberg.md: http://gerrit.cloudera.org:8080/#/c/21929/2/nikola_site_generator/posts/impalas-living-on-iceberg.md@2 PS2, Line 2: l The 'L' needs to be capitalized. http://gerrit.cloudera.org:8080/#/c/21929/2/nikola_site_generator/posts/impalas-living-on-iceberg.md@2 PS2, Line 2: Impalas The word "Impalas" is plural. I think this word should be "Impala's" unless the title is intended to refer to multiple Impala instances. -- To view, visit http://gerrit.cloudera.org:8080/21929 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: asf-site Gerrit-MessageType: comment Gerrit-Change-Id: I3beb67c6c209443703cbf3e9521c8b9f662ef733 Gerrit-Change-Number: 21929 Gerrit-PatchSet: 2 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Gabor Kaszab <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]> Gerrit-Reviewer: Noemi Pap-Takacs <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Comment-Date: Thu, 17 Oct 2024 18:58:37 +0000 Gerrit-HasComments: Yes
