You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@flink.apache.org by "zoltar9264 (via GitHub)" <gi...@apache.org> on 2023/05/20 17:03:27 UTC

[GitHub] [flink] zoltar9264 opened a new pull request, #22613: [FLINK-32141][checkpointing] turn down a log level in the SharedState…

zoltar9264 opened a new pull request, #22613:
URL: https://github.com/apache/flink/pull/22613

   …Registry
   
   ## What is the purpose of the change
   
   Turn down a log level in the SharedStateRegistry, described in  [FLINK-32141](https://issues.apache.org/jira/browse/FLINK-24611).


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] reswqa commented on pull request #22613: [FLINK-32141][checkpointing] turn down a log level in the SharedState…

Posted by "reswqa (via GitHub)" <gi...@apache.org>.
reswqa commented on PR #22613:
URL: https://github.com/apache/flink/pull/22613#issuecomment-1557380790

   > but I just checked the 1.17 code and found no conflicts (this commit can be merge into 1.17 directly), am I missing something? 
   
   I just found the fix version on the ticket is `1.17.2`, so I would like you to confirm if this need to pick it to 1.17. 
   
   Thank you for your confirmation, I will manually merge it :)


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] zoltar9264 commented on pull request #22613: [FLINK-32141][checkpointing] turn down a log level in the SharedState…

Posted by "zoltar9264 (via GitHub)" <gi...@apache.org>.
zoltar9264 commented on PR #22613:
URL: https://github.com/apache/flink/pull/22613#issuecomment-1557356094

   Thanks @reswqa for helping merge to master.
   
   > Can you open a PR and backport it to release-1.17.
   
   Sure, but I just checked the 1.17 code and found no conflicts (this commit can be merge into 1.17 directly), am I missing something?


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] fredia commented on pull request #22613: [FLINK-32141][checkpointing] turn down a log level in the SharedState…

Posted by "fredia (via GitHub)" <gi...@apache.org>.
fredia commented on PR #22613:
URL: https://github.com/apache/flink/pull/22613#issuecomment-1557146142

   @zoltar9264 Thanks for the PR, +1 for "FsStateChangelogStorage will directly use the FileStateHandle of the previous checkpoint instead of PlaceholderStreamStateHandle."  When changelog is enabled, `SharedStateRegistry` would print too much info log.


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] flinkbot commented on pull request #22613: [FLINK-32141][checkpointing] turn down a log level in the SharedState…

Posted by "flinkbot (via GitHub)" <gi...@apache.org>.
flinkbot commented on PR #22613:
URL: https://github.com/apache/flink/pull/22613#issuecomment-1555953469

   <!--
   Meta data
   {
     "version" : 1,
     "metaDataEntries" : [ {
       "hash" : "97ae387dfe466d2d1e32f10bfd4f62038e0b2ee4",
       "status" : "UNKNOWN",
       "url" : "TBD",
       "triggerID" : "97ae387dfe466d2d1e32f10bfd4f62038e0b2ee4",
       "triggerType" : "PUSH"
     } ]
   }-->
   ## CI report:
   
   * 97ae387dfe466d2d1e32f10bfd4f62038e0b2ee4 UNKNOWN
   
   <details>
   <summary>Bot commands</summary>
     The @flinkbot bot supports the following commands:
   
    - `@flinkbot run azure` re-run the last Azure build
   </details>


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] reswqa commented on pull request #22613: [FLINK-32141][checkpointing] turn down a log level in the SharedState…

Posted by "reswqa (via GitHub)" <gi...@apache.org>.
reswqa commented on PR #22613:
URL: https://github.com/apache/flink/pull/22613#issuecomment-1557196204

   Merged into master, @zoltar9264 Can you open a PR and backport it to `release-1.17`. Thanks!


-- 
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: issues-unsubscribe@flink.apache.org

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

[GitHub] [flink] reswqa merged pull request #22613: [FLINK-32141][checkpointing] turn down a log level in the SharedState…

Posted by "reswqa (via GitHub)" <gi...@apache.org>.
reswqa merged PR #22613:
URL: https://github.com/apache/flink/pull/22613


-- 
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: issues-unsubscribe@flink.apache.org

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