industrial=sugar_refinery is a defined value for a type of industrial facility
no, it is not, it is a tag specifying landuse, it means this land is used by sugar refineries. IMHO this is overly specific for landuse specification, I‘d rather use a scheme that describes actual refineries as features and not use landuse tagging at this level of specificity.
IMHO we should encourage works=sugar_refinery