Skip to content

ARROW-11782: [GLib][Ruby][Dataset] Remove bindings for internal classes - #10533

Closed
kou wants to merge 5 commits into
apache:masterfrom
kou:glib-dataset-factory
Closed

ARROW-11782: [GLib][Ruby][Dataset] Remove bindings for internal classes#10533
kou wants to merge 5 commits into
apache:masterfrom
kou:glib-dataset-factory

Conversation

@kou

@koukou commented Jun 15, 2021

Copy link
Copy Markdown
Member

No description provided.

@github-actions

Copy link
Copy Markdown

static Result<std::shared_ptr<DatasetFactory>> Make(
std::shared_ptr<fs::FileSystem> filesystem, const std::vector<fs::FileInfo>& files,
std::shared_ptr<FileFormat> format, FileSystemFactoryOptions options);

@koukouJun 15, 2021

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

@bkietz Can we export this factory method? I want to use this in Apache Arrow Dataset GLib.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If it's convenient for you I think it's fine

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

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

Thanks!

@koukou changed the title ARROW-11782: [GLib][Dataset] Remove bindings for internal classesARROW-11782: [GLib][Ruby][Dataset] Remove bindings for internal classesJun 16, 2021
@kou

kou commented Jun 16, 2021

Copy link
Copy Markdown
MemberAuthor

+1

@koukou closed this in f8661e0Jun 16, 2021
@kou
kou deleted the glib-dataset-factory branch June 16, 2021 05:03
sjperkins pushed a commit to sjperkins/arrow that referenced this pull request Jun 23, 2021
Closesapache#10533 from kou/glib-dataset-factory
Authored-by: Sutou Kouhei <kou@clear-code.com>
Signed-off-by: Sutou Kouhei <kou@clear-code.com>
Sign up for freeto 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.

2 participants

@kou@bkietz