Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add arbitrary fill value support #1666

Merged
merged 1 commit into from
Jun 8, 2020
Merged

Conversation

stavrospapadopoulos
Copy link
Member

Enable the user to set arbitrary fill values to attributes. The fill values are used when empty attribute values are retrieved during a read query. This can happen either (1) in the case of a dense array with "empty" regions, or (2) any array with missing attributes. Note that the latter (i.e., enabling writes with a subset of attributes) will be supported in a subsequent PR.

Breaking behavior: Now reading completely empty dense arrays will return fill values, whereas before it was returning zero results.

…values are used when empty attribute values are retrieved during a read query. This can happen either (1) in the case of a dense array with "empty" regions, or (2) any array with missing attributes. Note that the latter (i.e., enabling writes with a subset of attributes) will be supported in a subsequent PR.
HISTORY.md Show resolved Hide resolved
Copy link
Contributor

@joe-maley joe-maley left a comment

Choose a reason for hiding this comment

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

LGTM -- is there an immediate motivation for this other than schema evolution support?

@chris-allan
Copy link
Contributor

Mentioned this on TileDB/TileDB-Java#177. The current inconsistent defaults in tiledb/sm/misc/constants.cc feel wrong. They were changed as part of b88b34b some time ago. Not sure what the rationale was there.

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

Successfully merging this pull request may close these issues.

4 participants