Re: [PR] fix: set the result block to the decompressed data [kudu]

2024-07-19 Thread via GitHub
alexeyserbin commented on PR #51: URL: https://github.com/apache/kudu/pull/51#issuecomment-2239772973 Haven't heard from the author for more than one year after request to post the patch using gerrit. Closing this pull request for now. -- This is an automated message from the Apache Git

Re: [PR] fix: set the result block to the decompressed data [kudu]

2024-07-19 Thread via GitHub
alexeyserbin closed pull request #51: fix: set the result block to the decompressed data URL: https://github.com/apache/kudu/pull/51 -- 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 comm

Re: [PR] fix: set the result block to the decompressed data [kudu]

2024-07-19 Thread via GitHub
alexeyserbin commented on code in PR #51: URL: https://github.com/apache/kudu/pull/51#discussion_r1684709521 ## src/kudu/cfile/cfile_reader.cc: ## @@ -546,7 +546,7 @@ Status CFileReader::ReadBlock(const IOContext* io_context, scratch.Swap(&decompressed_scratch); // S

Re: [PR] logging: We should initialized logging BEFORE InstallLoggingCallback,… [kudu]

2024-07-19 Thread via GitHub
alexeyserbin closed pull request #19: logging: We should initialized logging BEFORE InstallLoggingCallback,… URL: https://github.com/apache/kudu/pull/19 -- 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

Re: [PR] move configuration constants into separate class and make public [kudu]

2024-07-19 Thread via GitHub
alexeyserbin commented on PR #2: URL: https://github.com/apache/kudu/pull/2#issuecomment-2239741000 mapreduce has been obsoleted a few years ago and removed from Kudu repo with f8d060bd1d9066b0a6d4afa9bd7eb4d4f082037f. So, closing this pull request. -- This is an automated message from t

Re: [PR] move configuration constants into separate class and make public [kudu]

2024-07-19 Thread via GitHub
alexeyserbin closed pull request #2: move configuration constants into separate class and make public URL: https://github.com/apache/kudu/pull/2 -- 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 s

Re: [PR] [doc](readme)Enhancement in link [kudu]

2024-07-19 Thread via GitHub
alexeyserbin closed pull request #52: [doc](readme)Enhancement in link URL: https://github.com/apache/kudu/pull/52 -- 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 unsubscrib

Re: [PR] Fixed Spelling. [kudu]

2024-07-19 Thread via GitHub
alexeyserbin closed pull request #4: Fixed Spelling. URL: https://github.com/apache/kudu/pull/4 -- 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-unsu