Skip to content
This repository was archived by the owner on Aug 23, 2020. It is now read-only.

Feature: Cache in between Tangle and database #1520

Closed
wants to merge 15 commits into from

Conversation

kwek20
Copy link
Contributor

@kwek20 kwek20 commented Jul 15, 2019

Description

Adds a cache that keeps transaction data in memory. This is required because the RocksDB cache does not work currently.

Fixes #935

Type of change

  • Enhancement (a non-breaking change which adds functionality)

How Has This Been Tested?

Added unit tests, ran on network.

Checklist:

  • [X My code follows the style guidelines for this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@kwek20 kwek20 requested a review from GalRogozinski July 15, 2019 22:17
@@ -1,19 +1,38 @@
package com.iota.iri.controllers;

import com.iota.iri.model.*;
import com.iota.iri.model.persistables.*;
import java.util.ArrayList;
Copy link
Contributor Author

@kwek20 kwek20 Jul 15, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whoeps i reformatted this by accident. Lets merge fast to not get too many conflicts :)

Changes are from line 150-180

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Higher level cache above the standard RocksDB cache
1 participant