Checked on YTB and all that I found was that ghost shader made by instancing the player trail every frame. This seems bad, as it can probally affect the game performance.
Performance-wise, it's actually fine to draw a sprite multiple times. In fact, it can be faster than a complex shader. Godot uses 2D batching, so the sprites are drawn using a single draw call, even if they use different modulate colors.