You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@phoenix.apache.org by "Nihal Jain (Jira)" <ji...@apache.org> on 2025/08/02 09:10:00 UTC
[jira] [Comment Edited] (PHOENIX-6851) Use spotless to format code in phoenix
[ https://issues.apache.org/jira/browse/PHOENIX-6851?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18011033#comment-18011033 ]
Nihal Jain edited comment on PHOENIX-6851 at 8/2/25 9:09 AM:
-------------------------------------------------------------
|| ||JIRA||master||5.2||5.1||
|1|PHOENIX-7441| ✅
[22054c2|https://github.com/apache/phoenix/commit/22054c24932804744e0f4916cc614b2a81c65667]| ✅
[c7063dd|https://github.com/apache/phoenix/commit/c7063dddac5efab0624a62cff62ef8478bb5b3cd]| ✅
[48364cf|https://github.com/apache/phoenix/commit/48364cf14b9bac28df91673dc8eb6f8445728322]|
|2|PHOENIX-7675| ✅
[571d458|https://github.com/apache/phoenix/commit/571d45844a4eac2914e8bda8269bc5aab1ebb665]
[{{67d47a6}}|https://github.com/apache/phoenix/commit/67d47a612e745cf7f21a8522eb92209c4c43784f] | ❌| ❌|
|3|PHOENIX-7676| ✅
[{{861010d}}|https://github.com/apache/phoenix/commit/861010d5cf8d18930a948c7c5dd1cc7be994a975]| ❌| ❌|
|4|PHOENIX-7442| ✅
[67d47a6|https://github.com/apache/phoenix/commit/67d47a612e745cf7f21a8522eb92209c4c43784f]| ❌| ❌|
|5|PHOENIX-7443| ✅
[e8ff86e|https://github.com/apache/phoenix/commit/e8ff86e6235f8bd65892d9126d797d497f3cf978]| ❌| ❌|
Plan to fix these starting tomorrow!
CC: [~stoty], [~vjasani]
was (Author: nihaljain.cs):
|| ||JIRA||master||5.2||5.1||
|1|PHOENIX-7441| ✅
[22054c2|https://github.com/apache/phoenix/commit/22054c24932804744e0f4916cc614b2a81c65667]| ✅
[c7063dd|https://github.com/apache/phoenix/commit/c7063dddac5efab0624a62cff62ef8478bb5b3cd]| ✅
[48364cf|https://github.com/apache/phoenix/commit/48364cf14b9bac28df91673dc8eb6f8445728322]|
|2|PHOENIX-7675| ✅
[73044e6|https://github.com/apache/phoenix/commit/73044e6f71465a148f6ffeb580c0a8912246cb44]
[571d458|https://github.com/apache/phoenix/commit/571d45844a4eac2914e8bda8269bc5aab1ebb665]| ❌| ❌|
|3|PHOENIX-7676| ✅
[{{861010d}}|https://github.com/apache/phoenix/commit/861010d5cf8d18930a948c7c5dd1cc7be994a975]| ❌| ❌|
|4|PHOENIX-7442| ✅
[67d47a6|https://github.com/apache/phoenix/commit/67d47a612e745cf7f21a8522eb92209c4c43784f]| ❌| ❌|
|5|PHOENIX-7443| ✅
[e8ff86e|https://github.com/apache/phoenix/commit/e8ff86e6235f8bd65892d9126d797d497f3cf978]| ❌| ❌|
Plan to fix these starting tomorrow!
CC: [~stoty], [~vjasani]
> Use spotless to format code in phoenix
> --------------------------------------
>
> Key: PHOENIX-6851
> URL: https://issues.apache.org/jira/browse/PHOENIX-6851
> Project: Phoenix
> Issue Type: Improvement
> Reporter: Aman Poonia
> Assignee: Nihal Jain
> Priority: Major
> Fix For: 5.3.0
>
>
> Similar to what Apache HBase does today, we want to similarly use spotless plugin to format code in Apache Phoenix.
> The goal is to improve code consistency across the project and make it easier for new contributors to onboard as today it is very difficult for new contributors (even exitsing one's) to ensure code is properly formatted across the repository in a consistent manner.
> Hence to overcome this hindrance, here is a summary of the proposed plan based on recent discussions and past experiences:
> # Integrate the Spotless plugin and update the code template
> ** Integrate the Spotless plugin into the Phoenix project. This will allow us to standardize code formatting through Maven, reducing reliance on IDE-specific formatting rules.
> ** Update the development template (dev/PhoenixCodeTemplate.xml) to sync the formatting standards used in HBase.
> ** Align the license header with the one used by HBase for consistency. 2) Apply Spotless to reformat the entire codebase:
> # Apply the Spotless plugin to reformat the entire codebase
> ** This will be done in a single commit to minimize noise in future patches.
> ** No manual changes should be made in this PR, for ease of review
> # Add Spotless to the pre-commit checks
> ** Integrate Spotless checks into the pre-commit process to ensure code consistency moving forward.
> ** Build is supposed to fail if code does not meet spotless rules
> ** We may want to add this step only after the initial reformatting is complete to avoid build failures and ease of backport to other branches.
> Hence we plan to create 3 separate JIRA sub-tasks to handle each of above mentioned steps independently and repeat for all active branches i.e 5.1, 5.2 and master.
> We should merge this to 5.x only after 5.2.1 is released.
> See [https://lists.apache.org/thread/9gbh5xs544h4q4w4xt64oy9osy8g8f81] for complete discussion on this issue.
> CC: [~stoty], [~mnpoonia], [~larsfrancke]
--
This message was sent by Atlassian Jira
(v8.20.10#820010)