https://github.com/thinkaurelius/titan
Revision baf297da41c887a167abc9c9141f25d7993d9ff4 authored by Dan LaRocque on 28 April 2015, 23:01:24 UTC, committed by Dan LaRocque on 28 April 2015, 23:01:24 UTC
The only substantial change in this commit is narrowing one condition in KCVSLogManager's constructor. Prior to this commit, the condition assumed that StoreFeatures.hasCellTTL implied !StoreFeatures.hasStoreTTL. This is true for all of the storage backends we have today, and even if it weren't true, it wouldn't result in incorrect operation, but it's still not necessarily true in general. Changed condition to check for StoreFeatures.hasCellTTL && !StorFeatures.hasStoreTTL, which was the original intent. The other changes are just cosmetic: * Added some comments * Removed a redundant precondition in TTLKCVSManager's constructor * Renamed TTLKCVSManager's supportsStoreTTL method to supportsAnyTTL. This is better because the method's return value is actually cellTTL || storeTTL... it's confusing to name that supportsStoreTTL when it really means "supportsCellOrStoreTTL".
1 parent 2c2a86d
Tip revision: baf297da41c887a167abc9c9141f25d7993d9ff4 authored by Dan LaRocque on 28 April 2015, 23:01:24 UTC
TTL: method rename, condition tweak, add comments
TTL: method rename, condition tweak, add comments
Tip revision: baf297d
| File | Mode | Size |
|---|---|---|
| docs | ||
| titan-all | ||
| titan-berkeleyje | ||
| titan-cassandra | ||
| titan-core | ||
| titan-dist | ||
| titan-doc | ||
| titan-es | ||
| titan-hadoop-parent | ||
| titan-hbase-parent | ||
| titan-lucene | ||
| titan-rexster | ||
| titan-solr | ||
| titan-test | ||
| .gitignore | -rw-r--r-- | 1.1 KB |
| CHANGELOG.asc | -rw-r--r-- | 25.0 KB |
| LICENSE.txt | -rw-r--r-- | 11.1 KB |
| NOTICE.txt | -rw-r--r-- | 5.8 KB |
| README.textile | -rw-r--r-- | 2.0 KB |
| RELEASING.md | -rw-r--r-- | 3.5 KB |
| TESTING.md | -rw-r--r-- | 3.7 KB |
| UPGRADE.asc | -rw-r--r-- | 16.6 KB |
| jre6.xslt | -rw-r--r-- | 1.5 KB |
| pom.xml | -rw-r--r-- | 69.5 KB |
Computing file changes ...
