[HBASE-23234] Provide .editorconfig based on checkstyle configuration

• Type: Task
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 3.0.0-alpha-1, 2.3.0, 1.6.0
• Component/s: build, tooling

Release Note

Adds a .editorconfig file with configurations populated by IntelliJ, based on our checkstyle configuration. There's lots of IntelliJ-specific configs in here that I assume are not replicated to Eclipse or Netbeans users. Any devs using those tools should push whatever updates they see fit, but please start with the checkstyle configs as the origin of truth.

• Assignee: Nick Dimiduk
• Reporter: Nick Dimiduk
• Created: 30/Oct/19 17:11
• Updated: 22/Nov/19 14:05
• Resolved: 21/Nov/19 19:26

Description

I don't have an exhaustive analysis of the issue, but there's at least one case where check style plugin configuration disagrees with our settings in dev-support/hbase_eclipse_formatter.xml.

Formatter settings produce this code chunk

      uncaughtExceptionHandler =
          (t, e) -> abort("Uncaught exception in executorService thread " + t.getName(), e);

but check style wants

      uncaughtExceptionHandler =
        (t, e) -> abort("Uncaught exception in executorService thread " + t.getName(), e);

Issue Links

• links to: GitHub Pull Request #846
• links to: GitHub Pull Request #861
• links to: GitHub Pull Request #862
• links to: https://github.com/checkstyle/checkstyle/issues/7236