ByteYue commented on PR #14425: URL: https://github.com/apache/doris/pull/14425#issuecomment-1321163030
> This PR will print too much logs. If you want to add log to debug temporarily, I suggest to commit in your own repo and test your own branch. Sorry for so many annoying logs. I do agree it's too many logs and some of them might be redundant. And I do want to seek some help from reviews about which log should be added😂. I intend to add some useful logs like 1. printing the rows processed by add_batch 2. _pending_batches_bytes when closing the node_channel 3. skipped tablet id when add batch 4. print the afterwards log when it's the last rpc 5. print the logic that has something to do with eos. The intention to add these logs is that recently we are encountered with one strange bug that one txn's logs show there were no delta writers for a node channel on one client BE. But it's curious cause when you open one node channel there must be at least one tablet and corresponding delta writer. And the afterwards load channel destructor log even show there is no memory usage indicating there is no write. It's so strange. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
