Update surround_substring to surround all substrings

[MrPowers]

It works properly when there is only one substring like here: surround_substring("spark rdd stuff", "rdd", "**", "||").

It does not work properly when there are two substrings, like here: surround_substring("spark rdd stuff with rdd", "rdd", "**", "||").

[SemyonSinchenko]

@MrPowers What do you think about the case when we want to branch depending on SparkConnect? For example:

if hasattr(spark, "_jvm"):
    spark_code(...)
else:
    spark_connect_code(...)

[MrPowers]

@SemyonSinchenko - the purpose of this code is to run on other libraries and identify any files that contain potentially problematic code.

I am considering creating a separate project with the intention of making Spark Connect less breaking. That library would use the design pattern you outlined above. It would be great if we could monkey patch in the "old way" and just do the new "Spark Connect way" under the hood (when possible of course).