You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@orc.apache.org by GitBox <gi...@apache.org> on 2022/11/06 05:27:13 UTC

[GitHub] [orc] dongjoon-hyun commented on pull request #1305: ORC-1306: Fixed indented code style for Java modules

dongjoon-hyun commented on PR #1305:
URL: https://github.com/apache/orc/pull/1305#issuecomment-1304722250

   Thank you for making a backport, @guiyanakuang . There exist a few case where `break` is indented in the same level with the next `case` statement in this PR. It looks weird to me. I guess we can fix them by eliminating the redundant `{}`.
   ```
   break;
   case ...
   ```


-- 
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-unsubscribe@orc.apache.org

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