[HUDI-4620] No expected exception is thrown when create hudi table without primaryKey

[watermelon12138]

Change Logs

No expected exception is thrown when create hudi table without primaryKey.

Expected exception:
“java.lang.IllegalArgumentException: No primaryKey is specified."

Thrown exception now:
“Can't find primaryKey uuid in root”

Impact

Describe any public API or user-facing feature change or any performance impact.

Risk level: none | low | medium | high

Choose one. If medium or high, explain what verification was done to mitigate the risks.

Contributor's checklist

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

[hudi-bot]

CI report:

• 952bef8 Azure: SUCCESS

Bot commands

@hudi-bot supports the following commands:

• @hudi-bot run azure re-run the last Azure build

[yihua]

Shall we also remove the default value in KeyGeneratorOptions so that Spark SQL has consistent behavior with Spark datasource?

public static final ConfigProperty<String> RECORDKEY_FIELD_NAME = ConfigProperty
      .key("hoodie.datasource.write.recordkey.field")
      .defaultValue("uuid")
      .withDocumentation("Record key field. Value to be used as the `recordKey` component of `HoodieKey`.\n"
          + "Actual value will be obtained by invoking .toString() on the field value. Nested fields can be specified using\n"
          + "the dot notation eg: `a.b.c`");

cc @YannByron @nsivabalan

[yihua]

Hi @watermelon12138, I just noticed that this change is already covered by #5771. I'll close this one as a duplicate. Feel free to review the other PR.