This is an automated email from the ASF dual-hosted git repository.

azagrebin pushed a commit to branch asf-site
in repository https://gitbox.apache.org/repos/asf/flink-web.git


The following commit(s) were added to refs/heads/asf-site by this push:
     new e4f85d6  [FLINK-13804][code style][zh] Set the initial capacity for a 
collection only if there is a good proven reason
e4f85d6 is described below

commit e4f85d62998c0aaaa8c4acb66604c4235944279a
Author: Andrey Zagrebin <azagre...@gmail.com>
AuthorDate: Wed Aug 28 11:41:04 2019 +0200

    [FLINK-13804][code style][zh] Set the initial capacity for a collection 
only if there is a good proven reason
---
 contributing/code-style-and-quality-java.zh.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/contributing/code-style-and-quality-java.zh.md 
b/contributing/code-style-and-quality-java.zh.md
index 335e889..f0de4e7 100644
--- a/contributing/code-style-and-quality-java.zh.md
+++ b/contributing/code-style-and-quality-java.zh.md
@@ -63,6 +63,7 @@ title:  "Apache Flink Code Style and Quality Guide — Java"
     * `contains()` before `get()` → `get()` and check null
     * `contains()` before `put()` → `putIfAbsent()` or `computeIfAbsent()`
     * Iterating over keys, getting values → iterate over `entrySet()`
+* **Set the initial capacity for a collection only if there is a good proven 
reason** for that, otherwise do not clutter the code. In case of **Maps** it 
can be even deluding because the Map's load factor effectively reduces the 
capacity.
 
 
 ### Lambdas

Reply via email to