Skip to content

Commit

Permalink
No longer rely on now-deprecated dump() for fragment info (#753)
Browse files Browse the repository at this point in the history
* No longer rely on now-deprecated dump() for fragment info

* Update NEWS, roll micro release [ci skip]
  • Loading branch information
eddelbuettel authored Sep 9, 2024
1 parent 56fd6aa commit 81847b3
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 2 deletions.
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
Package: tiledb
Type: Package
Version: 0.29.0.7
Version: 0.29.0.8
Title: Modern Database Engine for Complex Data Based on Multi-Dimensional Arrays
Authors@R: c(person("TileDB, Inc.", role = c("aut", "cph")),
person("Dirk", "Eddelbuettel", email = "dirk@tiledb.com", role = "cre"))
Expand Down
2 changes: 2 additions & 0 deletions NEWS.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,8 @@

* `NDRectangle` objects can now return their number of dimensions and dimension data types (#743)

* `FragmentInfo` objects are dump via the `<<` stringstream operator instead of a now-deprecated `dump()` method (#753)

## Documentation

* The documentation website now uses favicon symbols for all pages rendered (#739)
Expand Down
8 changes: 7 additions & 1 deletion src/libtiledb.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -4942,7 +4942,13 @@ std::string libtiledb_fragment_info_to_vacuum_uri(XPtr<tiledb::FragmentInfo> fi,
// [[Rcpp::export]]
void libtiledb_fragment_info_dump(XPtr<tiledb::FragmentInfo> fi) {
check_xptr_tag<tiledb::FragmentInfo>(fi);
return fi->dump();
#if TILEDB_VERSION >= TileDB_Version(2,27,0)
std::stringstream ss;
ss << *fi;
Rcpp::Rcout << ss.str();
#else
fi->dump();
#endif
}

// [[Rcpp::export]]
Expand Down

0 comments on commit 81847b3

Please sign in to comment.