[ https://issues.apache.org/jira/browse/DRILL-8474?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17841663#comment-17841663 ]
ASF GitHub Bot commented on DRILL-8474: --------------------------------------- cgivre commented on code in PR #2909: URL: https://github.com/apache/drill/pull/2909#discussion_r1582202511 ########## exec/vector/src/main/java/org/apache/drill/exec/record/metadata/MapBuilder.java: ########## @@ -185,6 +192,26 @@ public MapBuilder resumeMap() { return (MapBuilder) parent; } + /** + * Depending on whether the parent is a schema builder or map builder + * we resume appropriately. + */ + @Override + public void resume() { + if (Objects.isNull(parent)) Review Comment: @mbeckerle I don't know why the checkstyle is telling you the wrong file, but here, you'll need braces as well as at line 203. ie: ```java if (parent instanceof MapBuilder) { resumeMap(); } ``` > Add Daffodil Format Plugin > -------------------------- > > Key: DRILL-8474 > URL: https://issues.apache.org/jira/browse/DRILL-8474 > Project: Apache Drill > Issue Type: New Feature > Affects Versions: 1.21.1 > Reporter: Charles Givre > Priority: Major > Fix For: 1.22.0 > > -- This message was sent by Atlassian Jira (v8.20.10#820010)