[
https://issues.apache.org/jira/browse/IGNITE-23326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17924505#comment-17924505
]
Roman Puchkovskiy commented on IGNITE-23326:
--------------------------------------------
The patch looks good to me
> Configuration parser allows duplicated keys
> -------------------------------------------
>
> Key: IGNITE-23326
> URL: https://issues.apache.org/jira/browse/IGNITE-23326
> Project: Ignite
> Issue Type: Bug
> Reporter: Ivan Bessonov
> Assignee: Philipp Shergalis
> Priority: Major
> Labels: ignite-3
> Time Spent: 1h 10m
> Remaining Estimate: 0h
>
> Currently, this code leads to no warnings or errors:
> {code:java}
> String configTemplate = "ignite {\n"
> + " \"network\": {\n"
> + " \"port\":{},\n"
> + " \"nodeFinder\":{\n"
> + " \"netClusterNodes\": [ {} ]\n"
> + " }\n"
> + " },\n"
> + " storage.profiles: {"
> + " " + DEFAULT_STORAGE_PROFILE + ".engine: aipersist, "
> + " " + DEFAULT_STORAGE_PROFILE + ".size: 2073741824 "
> + " },\n"
> + " storage.profiles: {"
> + " " + DEFAULT_STORAGE_PROFILE + ".engine: aipersist, "
> + " " + DEFAULT_STORAGE_PROFILE + ".size: 2073741824 " //
> Avoid page replacement.
> + " },\n"
> + " clientConnector: { port:{} },\n"
> + " rest.port: {},\n"
> + " raft.fsync = " + fsync()
> + "}"; {code}
> This behavior is confusing and error-prone for the end user, we shouldn't
> allow it
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
