accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubbsii <...@git.apache.org>
Subject [GitHub] accumulo pull request: ACCUMULO-4145 Eliminate Text wrapping of ta...
Date Mon, 15 Feb 2016 07:07:33 GMT
Github user ctubbsii commented on the pull request:

    https://github.com/apache/accumulo/pull/70#issuecomment-184089574
  
    For reference, String.intern() should not be used for Java 1.6 runtime, as that could
grow the PermGen space significantly. It could possibly replace the WeakHashMap in 1.7, because
intern storage has been moved to the main Java heap. It might not actually be needed at all,
since we've effectively eliminated all the extra objects and copies that were left around
with Text. And, in Java 1.8 update 30, the G1 GC has a String dedupe feature which may eliminate
the extra copies without needing to intern.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message