Skip to content

Commit 5f2d3b0

Browse files
panbingkundongjoon-hyun
authored andcommitted
[SPARK-48763][FOLLOWUP] Make dev/lint-scala error message more accurate
### What changes were proposed in this pull request? The pr is followuping #47157, to make `dev/lint-scala` error message more accurate. ### Why are the changes needed? After move from: `connector/connect/server` `connector/connect/common` to: `sql/connect/server` `sql/connect/common` Our error message in `dev/lint-scala` should be updated synchronously. eg: <img width="709" alt="image" src="https://github.com/apache/spark/assets/15246973/d749e371-7621-4063-b512-279d0690d573"> <img width="1406" alt="image" src="https://github.com/user-attachments/assets/ab681963-37f7-4f48-9458-61f591477365"> ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Pass GA. Manually test. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #47585 from panbingkun/SPARK-48763_FOLLOWUP_2. Authored-by: panbingkun <panbingkun@baidu.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
1 parent 6631abc commit 5f2d3b0

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

dev/lint-scala

+1-1
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ ERRORS=$(./build/mvn \
3636
)
3737

3838
if test ! -z "$ERRORS"; then
39-
echo -e "The scalafmt check failed on connect or connector/connect at following occurrences:\n\n$ERRORS\n"
39+
echo -e "The scalafmt check failed on sql/connect or connector/connect at following occurrences:\n\n$ERRORS\n"
4040
echo "Before submitting your change, please make sure to format your code using the following command:"
4141
echo "./build/mvn scalafmt:format -Dscalafmt.skip=false -Dscalafmt.validateOnly=false -Dscalafmt.changedOnly=false -pl sql/connect/common -pl sql/connect/server -pl connector/connect/client/jvm"
4242
exit 1

0 commit comments

Comments
 (0)