You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by "cshannon (via GitHub)" <gi...@apache.org> on 2023/03/03 15:26:04 UTC

[GitHub] [accumulo-website] cshannon opened a new pull request, #377: Updated 2.1.1 release notes

cshannon opened a new pull request, #377:
URL: https://github.com/apache/accumulo-website/pull/377

   Add https://github.com/apache/accumulo/pull/3205 to the list of minor improvements


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [accumulo-website] ctubbsii commented on a diff in pull request #377: Updated 2.1.1 release notes

Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on code in PR #377:
URL: https://github.com/apache/accumulo-website/pull/377#discussion_r1124627257


##########
_posts/release/2022-11-02-accumulo-2.1.1.md:
##########
@@ -42,6 +42,7 @@ Apache Accumulo 2.1.1 is a patch release of the 2.1.0 LTM line.
 * {% ghi 3023 %}, {% ghi 3053 %} Prevent excessive logging when an ExternalCompaction
   is cancelled for cause and around the ThreadPools
 * {% ghi 3057 %} Modified Monitor to direct DataTable errors to console
+* {% ghi 3205 %} Support writing cluster config parser output to a file

Review Comment:
   This is an internal tool... not really user facing. I'm not sure it's important enough to highlight in the release notes. But, the whole release notes will be reviewed before release... so it doesn't matter. Go ahead and merge.



-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [accumulo-website] cshannon commented on a diff in pull request #377: Updated 2.1.1 release notes

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon commented on code in PR #377:
URL: https://github.com/apache/accumulo-website/pull/377#discussion_r1124629140


##########
_posts/release/2022-11-02-accumulo-2.1.1.md:
##########
@@ -42,6 +42,7 @@ Apache Accumulo 2.1.1 is a patch release of the 2.1.0 LTM line.
 * {% ghi 3023 %}, {% ghi 3053 %} Prevent excessive logging when an ExternalCompaction
   is cancelled for cause and around the ThreadPools
 * {% ghi 3057 %} Modified Monitor to direct DataTable errors to console
+* {% ghi 3205 %} Support writing cluster config parser output to a file

Review Comment:
   Fair enough, I figured I'd add it just in case since it was a draft. As you said, can remove later.



-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [accumulo-website] cshannon merged pull request #377: Updated 2.1.1 release notes

Posted by "cshannon (via GitHub)" <gi...@apache.org>.
cshannon merged PR #377:
URL: https://github.com/apache/accumulo-website/pull/377


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [accumulo-website] ctubbsii commented on a diff in pull request #377: Updated 2.1.1 release notes

Posted by "ctubbsii (via GitHub)" <gi...@apache.org>.
ctubbsii commented on code in PR #377:
URL: https://github.com/apache/accumulo-website/pull/377#discussion_r1124634845


##########
_posts/release/2022-11-02-accumulo-2.1.1.md:
##########
@@ -42,6 +42,7 @@ Apache Accumulo 2.1.1 is a patch release of the 2.1.0 LTM line.
 * {% ghi 3023 %}, {% ghi 3053 %} Prevent excessive logging when an ExternalCompaction
   is cancelled for cause and around the ThreadPools
 * {% ghi 3057 %} Modified Monitor to direct DataTable errors to console
+* {% ghi 3205 %} Support writing cluster config parser output to a file

Review Comment:
   Yeah, but I guess my point is that because it's draft, you don't really need a code review for such trivial updates to the draft.



-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org