[GitHub] [kudu] gangliao opened a new pull request, #51: fix: set the result block to the decompressed data

2023-03-12 Thread via GitHub
gangliao opened a new pull request, #51: URL: https://github.com/apache/kudu/pull/51 I mean, this is a bug. -- 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,

[GitHub] [kudu] gangliao commented on a diff in pull request #51: fix: set the result block to the decompressed data

2023-03-12 Thread via GitHub
gangliao commented on code in PR #51: URL: https://github.com/apache/kudu/pull/51#discussion_r1133448252 ## src/kudu/cfile/cfile_reader.cc: ## @@ -546,7 +546,7 @@ Status CFileReader::ReadBlock(const IOContext* io_context, scratch.Swap(_scratch); // Set the result

[GitHub] [kudu] acelyc111 commented on pull request #51: fix: set the result block to the decompressed data

2023-03-12 Thread via GitHub
acelyc111 commented on PR #51: URL: https://github.com/apache/kudu/pull/51#issuecomment-1465537459 Hi @gangliao, Thank you for the pull request. The Kudu project doesn't merge new code through GitHub PRs; we use the [gerrit](https://gerrit.cloudera.org/q/status:open%20project:kudu)