Skip to content

[Zeppelin 796] Duplicated notebook names should not be allowed - #1536

Closed
ghost wants to merge 3 commits into
masterfrom
unknown repository
Closed

[Zeppelin 796] Duplicated notebook names should not be allowed#1536
ghost wants to merge 3 commits into
masterfrom
unknown repository

Conversation

@ghost

Copy link
Copy Markdown

What is this PR for?

When a notebook is created/cloned/imported the title/name of the new notebook should not be duplicate of the existing notebook's title/name

What type of PR is it?

Improvement

Todos

  • Task
    What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-796

How should this be tested?

Create a notebook with name 'Note', Notebook should allow user to create a note
When a user try to create/clone/import a notebook with name 'Note', Notebook should not allow user to create note.
Screenshots (if appropriate)

Screenshot of error message displayed during create/clone notebook
create

Screenshot of error message displayed during import notebook
import

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

@anthonycorbacho

Copy link
Copy Markdown
Contributor

I think its okay to have a duplicate name, if the noteId is different nothing bad will happen. What kind of case are you trying to solve here?

@ghost

Copy link
Copy Markdown
Author

@anthonycorbacho : Lets say if i create a note with name 'Note'. If zeppelin allows me to create a new note with name 'Note', then after sometime when i search for one of this note, I am going to get confused here.

@anthonycorbacho

Copy link
Copy Markdown
Contributor

I see your point, but let say i want to import a note and i have a note with the same name, it will not allow me to do that and i feel like its a pretty bad user experience here. What do you think?

@ghost

Copy link
Copy Markdown
Author

@anthonycorbacho : For import note function too, we will have same problem as i pointed out first. Anyway for import note too, any user can give a new name for importing notebook

@ghost

Copy link
Copy Markdown
Author

@ALL : Please review this PR and let me know your comments

@AhyoungRyu

Copy link
Copy Markdown
Contributor

@rajarajan-g There are some conflicts in here. Could you handle them? :)

@ghost

ghost commented Nov 9, 2016

Copy link
Copy Markdown
Author

Ping

@ghost

ghost commented Nov 9, 2016

Copy link
Copy Markdown
Author

Now i have resolved the conflicts. Please review

@ghost

Copy link
Copy Markdown
Author

Any updates ?

@Tagar

Tagar commented Apr 3, 2017

Copy link
Copy Markdown
Contributor

IMHO, automatically adding " (1)", " (2)" .. etc to a note name in case of duplicate name found
might be a better solution than disallowing import at all. A variation on this idea - adding " (import date/time)" suffix to note name would do too (only in case if dup name found).

It'll be then at user's discretion to rename notes appropriately - so they would still be allowed to import a note.

@asfgitasfgit closed this in c38a0a0May 9, 2018
asfgit pushed a commit that referenced this pull request May 9, 2018
close#83close#86close#125close#133close#139close#146close#193close#203close#246close#262close#264close#273close#291close#299close#320close#347close#389close#413close#423close#543close#560close#658close#670close#728close#765close#777close#782close#783close#812close#822close#841close#843close#878close#884close#918close#989close#1076close#1135close#1187close#1231close#1304close#1316close#1361close#1385close#1390close#1414close#1422close#1425close#1447close#1458close#1466close#1485close#1492close#1495close#1497close#1536close#1545close#1561close#1577close#1600close#1603close#1678close#1695close#1739close#1748close#1765close#1767close#1776close#1783close#1799
Sign up for freeto 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

@anthonycorbacho@AhyoungRyu@Tagar@raja-imaginea