Skip to content

Conversation

@xushiyan
Copy link
Member

Change Logs

Use spark3.2 profile as default dev profile.

Impact

affect dev setup. not affecting bundle artifacts.

Risk level

medium

Documentation Update

NA

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

@xushiyan xushiyan force-pushed the HUDI-3088-default-spark32 branch 3 times, most recently from a0946bd to 0cc95e7 Compare December 4, 2022 19:30
@xushiyan xushiyan added dependencies Dependency updates priority:blocker Production down; release blocker labels Dec 5, 2022
Comment on lines 259 to +265
} else if ("/exampleEvolvedSchemaDeleteColumn.avsc".equals(schemaPath)) {
assertNull(record.get("number"));
assertFalse(record.hasField("number"));
} else {
assertEquals(index, record.get("number"));
}
assertNull(record.get("added_field"));
// TODO temp disable
// assertNull(record.get("added_field"));
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

need to revisit this test case

Comment on lines -69 to -78
<groupId>org.mortbay.jetty</groupId>
<artifactId>*</artifactId>
</exclusion>
<exclusion>
<groupId>javax.servlet.jsp</groupId>
<artifactId>*</artifactId>
</exclusion>
<exclusion>
<groupId>javax.servlet</groupId>
<artifactId>*</artifactId>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not seen these dep from spark-core or spark-sql

Comment on lines 257 to +269
<groupId>org.apache.spark</groupId>
<artifactId>spark-streaming-kafka-0-10_${scala.binary.version}</artifactId>
<version>${spark.version}</version>
<exclusions>
<exclusion>
<groupId>org.apache.hadoop</groupId>
<artifactId>hadoop-client-api</artifactId>
</exclusion>
<exclusion>
<groupId>org.apache.hadoop</groupId>
<artifactId>hadoop-client-runtime</artifactId>
</exclusion>
</exclusions>
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this exclusion affects utilities bundle dependency as it is included. needs to revisit the impact

@xushiyan xushiyan force-pushed the HUDI-3088-default-spark32 branch 2 times, most recently from 904a044 to 0aab4e9 Compare December 8, 2022 08:15
@xushiyan xushiyan force-pushed the HUDI-3088-default-spark32 branch from 2d3c710 to 5c5a904 Compare December 12, 2022 09:46
@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@xushiyan
Copy link
Member Author

xushiyan commented Apr 13, 2023

Moving to #8445

@xushiyan xushiyan closed this Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants