lordgamez commented on code in PR #1631:
URL: https://github.com/apache/nifi-minifi-cpp/pull/1631#discussion_r1298417482


##########
libminifi/include/core/logging/LoggerConfiguration.h:
##########
@@ -98,15 +98,15 @@ class LoggerConfiguration {
    */
   void initialize(const std::shared_ptr<LoggerProperties> &logger_properties);
 
-  static std::unique_ptr<io::InputStream> getCompressedLog(bool flush = false) 
{
-    return getCompressedLog(std::chrono::milliseconds{0}, flush);
+  static std::vector<std::unique_ptr<io::InputStream>> getCompressedLogs(bool 
flush = false) {
+    return getCompressedLogs(std::chrono::milliseconds{0}, flush);

Review Comment:
   I think it's a good idea, unfortunately I tried doing it and it cannot be 
handled as a single stream because of the compression. As the segments are 
compressed individually when forwarding the input stream to the zlib 
decompressor it only returns the first segment decompressed, probably due to 
finding the end of the archive.



-- 
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-unsubscr...@nifi.apache.org

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

Reply via email to