Skip to content

ADR: HDFS Raster Layers - #1582

Merged
echeipesh merged 2 commits into
locationtech:masterfrom
echeipesh:adr-hdfs-layers
Aug 9, 2016
Merged

echeipesh merged 2 commits into
locationtech:masterfrom
echeipesh:adr-hdfs-layers

Conversation

@echeipesh

Copy link
Copy Markdown
Contributor

This ADR describes current implementation of the HDFS layers and key design decisions that were considered. This is intended to as both documentation and starting off point for further improvements.


In a single value query we are given an instance of `K` and we must produce a corresponding `V` or an error. The first step is to locate the `MapFile` which potentially contains `(K, V)` record. Because the layer records are indexed by their SFC index we map `K` to `i: Long` and determine which file contains potential match by examining the file listing and finding the file with maximum starting index that is less than equal `i`. At this point the `MapFile` must be opened and queried for the key.

Because both listing all the files in the layer directory and opening a single MapFile we cache these steps. The file listing needs to be done only once per `HadoopValueReader` instance and we maintain an LRU cache of open `MapFile`s. Because SFC preserves some spatial locality of the records, geographically close records are likely to be close in SFC index, and we expect key/value queries to be geographically grouped, for instance requests from a map viewer, the `MapFile` LRU cache can be expected to have very high hit-rate.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Because both listing all the files in the layer directory and opening a single MapFile we cache these steps. I feel like this is missing some words.

@lossyrob

Copy link
Copy Markdown
Member

+1 after changes.

@echeipesh
echeipesh merged commit a7c8b5d into locationtech:master Aug 9, 2016
@lossyrob lossyrob added this to the 1.0 milestone Oct 18, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants