You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/05/29 04:15:13 UTC

[GitHub] [incubator-kvrocks] git-hulk opened a new pull request, #613: Allow to send logs to stdout/stderr

git-hulk opened a new pull request, #613:
URL: https://github.com/apache/incubator-kvrocks/pull/613

   This closes #477 
   
   Currently, we only supported sending logs to the log file,
   but we expected to collect logs through stdout/stderr instead of
   the log file when deploying the Kvrocks in the container. We also
   upgrade the glog to v0.6.0 since logstostdout was supported after that.


-- 
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: dev-unsubscribe@kvrocks.apache.org

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


[GitHub] [incubator-kvrocks] git-hulk commented on pull request #613: Allow to send logs to stdout/stderr

Posted by GitBox <gi...@apache.org>.
git-hulk commented on PR #613:
URL: https://github.com/apache/incubator-kvrocks/pull/613#issuecomment-1140391629

   cool, thanks @PragmaTwice.


-- 
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: dev-unsubscribe@kvrocks.apache.org

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


[GitHub] [incubator-kvrocks] git-hulk commented on pull request #613: Allow to send logs to stdout/stderr

Posted by GitBox <gi...@apache.org>.
git-hulk commented on PR #613:
URL: https://github.com/apache/incubator-kvrocks/pull/613#issuecomment-1140376352

   This CI failure seems was related to issue: https://github.com/google/snappy/pull/128, will investigate why it only happens in current PR(maybe the GCC version was updated).


-- 
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: dev-unsubscribe@kvrocks.apache.org

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


[GitHub] [incubator-kvrocks] git-hulk merged pull request #613: Allow to send logs to stdout/stderr

Posted by GitBox <gi...@apache.org>.
git-hulk merged PR #613:
URL: https://github.com/apache/incubator-kvrocks/pull/613


-- 
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: dev-unsubscribe@kvrocks.apache.org

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


[GitHub] [incubator-kvrocks] git-hulk commented on pull request #613: Allow to send logs to stdout/stderr

Posted by GitBox <gi...@apache.org>.
git-hulk commented on PR #613:
URL: https://github.com/apache/incubator-kvrocks/pull/613#issuecomment-1140425688

   Thanks @ShooterIT @tisonkun @PragmaTwice, will merge this PR.


-- 
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: dev-unsubscribe@kvrocks.apache.org

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


[GitHub] [incubator-kvrocks] PragmaTwice commented on pull request #613: Allow to send logs to stdout/stderr

Posted by GitBox <gi...@apache.org>.
PragmaTwice commented on PR #613:
URL: https://github.com/apache/incubator-kvrocks/pull/613#issuecomment-1140390540

   > This CI failure seems was related to issue: [google/snappy#128](https://github.com/google/snappy/pull/128), will investigate why it only happens in current PR(maybe the GCC version was updated).
   
   Try add `BUILD_SHARED_LIBS=OFF` to https://github.com/apache/incubator-kvrocks/blob/unstable/cmake/glog.cmake#L32.
   Some options are changed in glog v0.6.0, which may affect the global configurations.


-- 
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: dev-unsubscribe@kvrocks.apache.org

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