Skip to content

Add putMetadata overload #156

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

Merged
merged 1 commit into from
May 4, 2020

Conversation

pomadchin
Copy link
Contributor

@pomadchin pomadchin commented May 4, 2020

This PR adds an extra putMetadata overload which is more user friendly.

It is the follow up PR for the #155

@gsvic
Copy link
Contributor

gsvic commented May 4, 2020

Thanks for the follow up @pomadchin! I recently opened a similar PR which returns a HashMap of Strings and Objects instead of NativeArrays to make the API more user-friendly, please take a look if you'd like: #157.

It seems that your PR contains some changes that mine does not, so we can somehow merge these two. + @Shelnutt2

@pomadchin pomadchin force-pushed the feature/metadata-overload branch from d2acf76 to 71f4e3c Compare May 4, 2020 13:25
@pomadchin
Copy link
Contributor Author

pomadchin commented May 4, 2020

@gsvic that is nice! you can close this PR and grab the putMetadata method definition from this one!

@pomadchin pomadchin force-pushed the feature/metadata-overload branch 2 times, most recently from 970fc44 to 28592dd Compare May 4, 2020 13:30
@pomadchin pomadchin changed the title Metadata methods overloads PutMetadata overload May 4, 2020
@pomadchin pomadchin force-pushed the feature/metadata-overload branch from 28592dd to d423d41 Compare May 4, 2020 13:32
@pomadchin pomadchin changed the title PutMetadata overload Add putMetadata overload May 4, 2020
@gsvic gsvic self-requested a review May 4, 2020 13:43
Copy link
Contributor

@gsvic gsvic left a comment

Choose a reason for hiding this comment

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

LGTM!

@gsvic gsvic requested a review from Shelnutt2 May 4, 2020 13:45
Copy link
Member

@Shelnutt2 Shelnutt2 left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution!

@Shelnutt2 Shelnutt2 merged commit 7c20b42 into TileDB-Inc:master May 4, 2020
@pomadchin pomadchin deleted the feature/metadata-overload branch May 4, 2020 14:37
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.

3 participants