Hello,
this is a good question.

For performance/efficiency reasons, in the OPC, it is recommended to use subscriptions, instead of periodically polling by Reads. The subscriptions allow the server to optimize the data collection, and the data transfer (only changes are sent).

So, this is a general rule, and I would normally follow it when designing a new solution. On the other hand, subscriptions have some disadvantages, too. Thet are more difficult to setup, and the code gets more complicated.

If you have actually positively determined that your solution works, and does not create too much load anywhere (the device, the communication to the device, the server, the network, the client etc.), there is nothing that prevents you from simply using what you have done, especially if that works reliably already. But your test must be with the full number of items, real amount of data, and in parallel with anything else that would run on these systems. If it works, then you are OK.

Regards