kaknikhil commented on a change in pull request #553:
URL: https://github.com/apache/madlib/pull/553#discussion_r582413471



##########
File path: 
src/ports/postgres/modules/deep_learning/madlib_keras_model_selection.sql_in
##########
@@ -873,9 +872,9 @@ CREATE OR REPLACE FUNCTION 
MADLIB_SCHEMA.generate_model_configs(
     object_table            VARCHAR DEFAULT NULL
 ) RETURNS VOID AS $$
     PythonFunctionBodyOnly(`deep_learning', `madlib_keras_model_selection')
-    with AOControl(False):

Review comment:
       If the user has appendonly set to true, then our code will fail if we 
don't have the `with aocontrol(False)` code. 
   You can reproduce this by setting `set 
gp_default_storage_options='appendonly=true';` and then running 
load_model_selection_table/generate_model_configs will error out with
   ```
   ERROR:  spiexceptions.FeatureNotSupported: append-only tables do not support 
unique indexes
   ```




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

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to