[SPARK-27193][SQL] CodeFormatter should format multiple comment lines correctly

[Ngone51]

What changes were proposed in this pull request?

when enable spark.sql.codegen.comments, there will be multiple comment lines. However, CodeFormatter can not handle multiple comment lines currently:

/* 001 */ public Object generate(Object[] references) {
/* 002 */   return new GeneratedIteratorForCodegenStage1(references);
/* 003 */ }
/* 004 */
/* 005 */ /**
 * Codegend pipeline for stage (id=1)
 * *(1) Project [(id#0L + 1) AS (id + 1)#3L]
 * +- *(1) Filter (id#0L = 1)
 *    +- *(1) Range (0, 10, step=1, splits=4)
 */
/* 006 */ // codegenStageId=1
/* 007 */ final class GeneratedIteratorForCodegenStage1 extends org.apache.spark.sql.execution.BufferedRowIterator {

After applying this pr:

/* 001 */ public Object generate(Object[] references) {
/* 002 */   return new GeneratedIteratorForCodegenStage1(references);
/* 003 */ }
/* 004 */
/* 005 */ /**
/* 006 */  * Codegend pipeline for stage (id=1)
/* 007 */  * *(1) Project [(id#0L + 1) AS (id + 1)#4L]
/* 008 */  * +- *(1) Filter (id#0L = 1)
/* 009 */  *    +- *(1) Range (0, 10, step=1, splits=2)
/* 010 */  */
/* 011 */ // codegenStageId=1
/* 012 */ final class GeneratedIteratorForCodegenStage1 extends org.apache.spark.sql.execution.BufferedRowIterator {

How was this patch tested?

Tested Manually.

[Ngone51]

ping @cloud-fan @mgaido91 @sameeragarwal

Can you have a look at this ? Thanks.

[mgaido91]

Not a big deal, but the change makes sense to me.

[SparkQA]

Test build #103607 has finished for PR 24133 at commit 71cb419.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds no public classes.

[cloud-fan]

thanks, merging to master!

[Ngone51]

thank you @mgaido91 @cloud-fan