Skip to content

[CURATOR-565] check useContainerParentsIfAvailable for ExistsBuilderImpl. - #354

Closed
PhantomThief wants to merge 1 commit into
apache:masterfrom
PhantomThief:CURATOR-565
Closed

PhantomThief wants to merge 1 commit into
apache:masterfrom
PhantomThief:CURATOR-565

Conversation

@PhantomThief

Copy link
Copy Markdown

No description provided.

@Randgalt

Copy link
Copy Markdown
Member

Please provide a test case and/or more information on why this is needed.

@eolivelli

Copy link
Copy Markdown
Contributor

@PhantomThief do you have cycles to follow up on this patch ?

@tisonkun

Copy link
Copy Markdown
Member

@PhantomThief this patch may make sense as we use different logic handle these flags between CreateBuilder and ExistBuilder. However, it may not be resolved in this way but align to how CreateBuilderImpl handles createParentsIfNeeded and createParentsAsContainers.

@tisonkun

tisonkun commented Sep 6, 2022

Copy link
Copy Markdown
Member

Closed as stale. Feel free to resubmit it if you're still willing to proceed.

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.

4 participants