shfshihuafeng commented on PR #2909:
URL: https://github.com/apache/drill/pull/2909#issuecomment-2081475241
> This fails its tests due to a maven checkstyle failure. It's complaining
about Drill:Exec:Vectors, which my code has no changes to.
>
> Can someone advise on what is wrong here?
/home/runner/work/drill/drill/exec/vector/src/main/java/org/apache/drill/exec/record/metadata/MapBuilder.java:201:5
you need add if' construct must use '{}',like following ?
if (Objects.isNull(parent)) {
throw new IllegalStateException("Call to resume() on MapBuilder with no
parent.");
}
> This fails its tests due to a maven checkstyle failure. It's complaining
about Drill:Exec:Vectors, which my code has no changes to.
>
> Can someone advise on what is wrong here?
exec/vector/src/main/java/org/apache/drill/exec/record/metadata/MapBuilder.java
201
i think you need add {} for if
```
if (Objects.isNull(parent)) {
throw new IllegalStateException("Call to resume() on MapBuilder with no
parent.");
}
```
--
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...@drill.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
