[jira] [Commented] (DRILL-8313) introduce configuration for yaml parsing to override the default max file size

["PJ Fanning (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/DRILL-8313?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17607120#comment-17607120 ] 

PJ Fanning commented on DRILL-8313:
-----------------------------------

[~dzamo] [~cgivre] When searching Drill code, I can find no direct use of snakeyaml in Drill. The only thing I found was:

drill-rdbms-metastore/pom.xml
{code:java}
<!-- Is needed to allow yaml parsing in JOOQ -->
<dependency>
  <groupId>org.yaml</groupId>
  <artifactId>snakeyaml</artifactId>
</dependency> {code}
Do you think we need to worry about yaml files that are larger than 3Mb here?

> introduce configuration for yaml parsing to override the default max file size
> ------------------------------------------------------------------------------
>
>                 Key: DRILL-8313
>                 URL: https://issues.apache.org/jira/browse/DRILL-8313
>             Project: Apache Drill
>          Issue Type: Improvement
>            Reporter: PJ Fanning
>            Priority: Major
>
> snakeyaml 1.32 brings in a default limit of 3Mb when parsing yaml files.
> Need to allow users to specify another value if they need to.
> [https://bitbucket.org/snakeyaml/snakeyaml/src/72dfa9f1074abe2b8a6c8776bee4476b0aed02e3/src/main/java/org/yaml/snakeyaml/LoaderOptions.java]
> I only became aware of this issue in the last few hours.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)