Skip to content
This repository has been archived by the owner on Jun 14, 2024. It is now read-only.

Update documentation - ZOrderCoveringIndex #522

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sezruby
Copy link
Collaborator

@sezruby sezruby commented Jan 14, 2022

What is the context for this pull request?

What changes were proposed in this pull request?

Update user document for ZOrderCoveringIndex

link to page: https://sezruby.github.io/hyperspace/docs/ug-index-types/

Does this PR introduce any user-facing change?

No, the PR only contains documentation update

How was this patch tested?

https://sezruby.github.io/hyperspace/docs/ug-index-types/

@sezruby sezruby added the documentation Improvements or additions to documentation label Jan 14, 2022
@sezruby sezruby self-assigned this Jan 14, 2022

As of v0.6, Hyperspace supports the following two covering indexes:

1. [(Bucketed) Covering Index](#covering-index)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not sure how we can name the covering index.. bucketed? shuffled? 🤔

title: "Index types"
permalink: /docs/ug-index-types/
excerpt: "Mutable dataset guide"
last_modified_at: 2021-12-29
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

nit: update the date

from hyperspace import Hyperspace
from hyperspace import CoveringIndexConfig
hs = Hyperspace(sparkSession)
idxConfig = CoveringIndexConfig('<indexName>', ['<indexedCol1>', '<indexedCol2>'], ['<includedCol1>'])
Copy link
Contributor

Choose a reason for hiding this comment

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

Using double quote will be more consistent with the following line. (And I personally prefer double quote 😄)
'<indexName>' -> "<indexName>"

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

Successfully merging this pull request may close these issues.

2 participants