amousavigourabi commented on PR #1203: URL: https://github.com/apache/parquet-mr/pull/1203#issuecomment-1828048073
> > @wgtmac , after the comments on the code style I had a quick look through the repository and found that indentations and the such differ quite drastically between (and even within) files. ParquetWriter has indents of four spaces for the constructor arguments, where ParquetReader has them at 14 spaces. Would you feel anything for a more extensive `.editorconfig` and easy-to-use linter such as spotless? > > Yes, I think that would be good to make the style consistent across all files automatically. I'll get started on expanding the `.editorconfig` for at least IntelliJ and setting up a compatible Spotless configuration then. -- 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-unsubscr...@parquet.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org