codope commented on code in PR #11610: URL: https://github.com/apache/hudi/pull/11610#discussion_r1673266828
########## website/versioned_docs/version-0.15.0/sql_ddl.md: ########## @@ -67,7 +67,10 @@ PARTITIONED BY (dt); ``` :::note -You can also create a table partitioned by multiple fields by supplying comma-separated field names. For, e.g., "partitioned by dt, hh" +You can also create a table partitioned by multiple fields by supplying comma-separated field names. +When creating a table partitioned by multiple fields, ensure that you specify the columns in the `PARTITIONED BY` clause +in the same order as they appear in the `CREATE TABLE` schema. For example, for the above table, the partition fields +should be specified as `PARTITIONED BY (dt, hh)`. Review Comment: I could not find sql ddl docs in earlier version but later realized this was part of quickstart itself. Tested back till 0.10, which error out, but other versions create table and insert data in incorrect partition. Updated the PR. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
