search

sisoputnfrba / 2018-1c-re-distinto

Enunciado del TP del 1c2018 "Re Distinto"
https://sisoputnfrba.gitbook.io/re-distinto/
2 stars 3 forks source link

Correcciones para la errata v2 #6

Closed iago64 closed 6 years ago

iago64 commented 6 years ago

Basado en master y el branch de errata2 Ajustes de ambigüedades y cierre de definiciones en:

afilgueira commented 6 years ago

Si, no había notado esto. Coincido en lo de dejar claro lo del bloqueo

El mar., 8 may. 2018 12:21, Tomás Ferraro notifications@github.com escribió:

@tferraro requested changes on this pull request.

Lo veo bastante sólido en gral. Solo dejé una nota sobre algo que me sigue haciendo ruido desde los cambios de la errata anterior :)

In coordinador.md https://github.com/sisoputnfrba/2018-1c-re-distinto/pull/6#discussion_r186765976 :

  1. Se elige la Instancia asociada y se le envía la solicitud.
    1. La instancia retorna al Coordinador
    2. El Coordinador logea la respuesta y envía al ESI

-En el caso que el coordinador decida en el paso 2 que la operación no puede ser ejecutada porque la instancia no existe más en el sistema, o porque el recurso al que intenta acceder se encuentra tomado(^7), le avisará al Planificador. El Planificador encolará el ESI en cuestión en una cola de bloqueados; en ambos casos haciendo referencia al recurso que se intentó bloquear. +En el caso que el coordinador decida en el paso 2 que la operación no puede ser ejecutada porque la instancia no existe más en el sistema, o porque el recurso al que intenta acceder se encuentra tomado(^9), le avisará al Planificador. El Planificador abortará al ESI en cuestión, y borrará la entrada trasgresora.

o porque el recurso al que intenta acceder se encuentra tomado(^9), le avisará al Planificador. El Planificador abortará al ESI en cuestión, y borrará la entrada trasgresora.

Lo mismo que dije en #5 https://github.com/sisoputnfrba/2018-1c-re-distinto/pull/5, si un recurso/clave se encuentra tomado, no queremos pasar a bloqueado? Porqué abortarlo? Me suena que volvió a haber una mezcla entre ambas frases.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/sisoputnfrba/2018-1c-re-distinto/pull/6#pullrequestreview-118409724, or mute the thread https://github.com/notifications/unsubscribe-auth/ABtDgtovuRV-aaVIyXvuzOVrvFwz_RtRks5twbfygaJpZM4T17yt .