uros-b commented on code in PR #57001:
URL: https://github.com/apache/spark/pull/57001#discussion_r3524871491


##########
common/utils/src/main/resources/error/error-conditions.json:
##########
@@ -8402,6 +8402,36 @@
           "Invalid partitioning: <cols> is missing or is in a map or array."
         ]
       },
+      "PIPELINE_DATASET_BUCKETING" : {
+        "message" : [
+          "Bucketing in a CREATE <pipelineDatasetType> statement. Remove the 
bucket specification from the statement."
+        ]
+      },
+      "PIPELINE_DATASET_COLUMN_CONSTRAINTS" : {
+        "message" : [
+          "Column constraints in a CREATE <pipelineDatasetType> statement. 
Remove the CHECK, UNIQUE, PRIMARY KEY and FOREIGN KEY constraints from the 
statement."
+        ]
+      },
+      "PIPELINE_DATASET_HIVE_SERDE" : {
+        "message" : [
+          "Hive SerDe format options in a CREATE <pipelineDatasetType> 
statement."

Review Comment:
   Nit: this is the only new condition omitting a remediation hint, so it's 
slightly less actionable than the other five. A one-liner like "Use the USING 
clause or a non-SerDe format instead." would align it better with the other 
ones.



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to