Tecnicamente todo bien aqui, el unico comentario es quizas usar nombres de campos que puedan ayudar a una persona externa entender la razon de ser de esa variable.
En este caso puntual, podria ser mas util nombrar la columna userId de la tabla note por createdBy o ownedBy; de esta forma queda explicito el proposito de esa columna.
Tecnicamente todo bien aqui, el unico comentario es quizas usar nombres de campos que puedan ayudar a una persona externa entender la razon de ser de esa variable.
En este caso puntual, podria ser mas util nombrar la columna
userIdde la tablanoteporcreatedByoownedBy; de esta forma queda explicito el proposito de esa columna.