do we need sql update here?