[
https://issues.apache.org/jira/browse/CASSANDRA-7736?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
T Jake Luciani updated CASSANDRA-7736:
--------------------------------------
Attachment: 7736.txt
Removed the non-required ones and left the ones on the ones used by the
NativeCell comparators.
This yield a ~5% gain for me.
> Clean-up, justify (and reduce) each use of @Inline
> --------------------------------------------------
>
> Key: CASSANDRA-7736
> URL: https://issues.apache.org/jira/browse/CASSANDRA-7736
> Project: Cassandra
> Issue Type: Improvement
> Components: Core
> Reporter: Benedict
> Assignee: T Jake Luciani
> Priority: Minor
> Fix For: 2.1.1
>
> Attachments: 7736.txt
>
>
> \@Inline is a delicate tool, and should in all cases we've used it (and use
> it in future) be accompanied by a comment justifying its use in the given
> context both theoretically and, preferably, with some brief description
> of/link to steps taken to demonstrate its benefit. We should aim to not use
> it unless we are very confident we can do better than the normal behaviour,
> as poor use can result in a polluted instruction cache, which can yield
> better results in tight benchmarks, but worse results in general use.
> It looks to me that we have too many uses already. I'll look over each one as
> well, and we can compare notes. If there's disagreement on any use, we can
> discuss, and if still there is any dissent should always err in favour of
> *not* using \@Inline.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
