Parece adequado, mas pode não ser. Você é a pessoa que mais conhece o problema real e tem dúvidas, imagine nós que não temos ideia onde isso será usado, de que forma, quais as necessidades de hoje e de amanhã.
O que eu quero dizer é que tanta coisa pode dar errada ou mudar, então pessoas aleatórias na internet tem menos condições de dizer se tudo dará certo do que você próprio. Tem que fazer o melhor possível e estar preparado para possíveis mudanças, não fazer nada que dificulte mudar depois. Isso é muito raro de se ver, aí reside o real problema.
Na verdade tenho muinhas dúvidas se precisa tudo isso. Me parece que na verdade algumas dessas tabelas tem relação 1:1 e nesse caso não vejo muita vantagem estar em tabela separada. Pra mim as tabelas ligadas ao SKU deveriam ser colunas simples na própria tabela, mesmo após o comentário abaixo não vejo motivo para essas tabelas existirem. Provavelmente eu simplificaria eliminando 5 tabelas. Mas quem sou eu pra dizer o que deve fazer, ainda pode ser que tenha algo que eu não sei, por isso fica difícil responder.
Eu tenho até dúvidas se deveria ter uma separação entre produto e SKU. Até vejo alguma utilidade, mas pode não ser necessário. Muitos sistemas possuem o que seria só o SKU, afinal se cada SKU é diferente, é outro produto e o que chama de produto talvez seja um grupo.
Acho estranho produto_tamanho
ter relação com o produto e com SKU, isso parece um erro.
Nem vou falar dos nomes confusos, aí é gosto. Nem tudo deve ser VARCHAR(45)
, isso é um problema que vai além da modelagem, ou seja, esse é um problema tão grande que fazendo assim qualquer modelagem "servirá". Esse tipo de coisa dificultará mudanças futuras. Mas eu imagino que o que mais dificultará nem dá para ver neste modelo.
Se a pergunta tivesse colocada como texto e não como imagem eu teria reformulado o modelo e postado aqui como eu faria.
Coloquei no GitHub para referência futura.