Parece estar tudo certo de acordo com as informações fornecidas. Claro que as regras do que pode ou não fazer, de como apresentar (incluindo links), será determinado pelo código (ainda que dentro do banco de dados, o que não me agrada) e não pela modelagem.
Eu só usaria um NULL
(parece que ele foi preparado para isto) ou valor 0 em posted_to
quando por uma pergunta. Não tem porque duplicar nesta coluna a informação que já está no product_id
.
E se eu entendi o que é o type
, talvez, ele nem seja necessário, porque dá para inferir pelo conteúdo de posted_to
. Mas não sei se vale fazer esta economia, até porque no futuro pode mudar alguma coisa e isto não ser mais possível. Claro que teria como resolver o problema facilmente se for necessário. Podemos pensar em YAGNIYAGNI aqui, ou não tentar ser muito clever (o problema é decidir o que é clever demais).
Um date_posted
ser NOT NULL
parece ser adequado também.
Aí poderia pensar em outras coisas que não estão contempladas nessa análise básica. Por exemplo: a possibilidade do comentário ser removido (mas mantê-lo no banco de dados) exigira uma coluna para indicar o estado.