Skip to content

Update movie.js#9

Closed
Ultron021122 wants to merge 1 commit intomidudev:mainfrom
Ultron021122:patch-1
Closed

Update movie.js#9
Ultron021122 wants to merge 1 commit intomidudev:mainfrom
Ultron021122:patch-1

Conversation

@Ultron021122
Copy link
Copy Markdown

Modificaciones en getAll y getById

Modificaciones en getAll y getById
@midudev
Copy link
Copy Markdown
Owner

midudev commented Apr 10, 2026

Gracias por la contribucinnn! Sin embargo, esta PR tiene dos problemas:

): el schema original usa (singular), cambiarlo romper la app para quien siga el curso oficial.
2. Regresinnn de rendimiento en getById: usar BIN_TO_UUID(id) = ? aplica una funcinnn a la columna, impidiendo que MySQL use el dice. La versinnn original con id = UUID_TO_BIN(?) es ms eficiente.

La implementacinnn de getByGenre es una buena idea, pero necesitar adaptarse al schema del curso.

@midudev midudev closed this Apr 10, 2026
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.

2 participants