Skip to content

Conversation

@cgiachalis
Copy link
Contributor

closes #779

before:

uri <- tempfile()
tiledb_group_create(uri)

grp <- tiledb_group(uri) # Defaults to READ
grp <- tiledb_group_close(grp)

tiledb_group_open(grp)   # Should default to READ
#> Error in type != "MODIFY_EXCLUSIVE" || tiledb_version(TRUE) >= "2.12.0": 'length = 3' in coercion to 'logical(1)'

after:

tiledb_group_open(grp)
#> filee887faf6ad3 GROUP

Copy link
Member

@mojaveazure mojaveazure left a comment

Choose a reason for hiding this comment

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

Hey @cgiachalis, thanks for catching this! Can you bump the development version in the DESCRIPTION and add a changelog item to NEWS.md before merging?

@cgiachalis
Copy link
Contributor Author

That's done from my end! We're good to go..

@mojaveazure mojaveazure merged commit ae107bc into TileDB-Inc:main Aug 12, 2025
7 checks passed
@mojaveazure
Copy link
Member

Awesome, thank you @cgiachalis!

@cgiachalis cgiachalis deleted the cg/fix-779 branch August 13, 2025 13:52
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.

tiledb_group_open() should default to 'READ'

2 participants