[GitHub] [flink] snuyanzin commented on pull request #22144: [FLINK-31102][table] Add ARRAY_REMOVE function.

["snuyanzin (via GitHub)" <gi...@apache.org>]

snuyanzin commented on PR #22144:
URL: https://github.com/apache/flink/pull/22144#issuecomment-1463428022

   thanks for the contribution
   can you please fix description?
   e.g.
     1. I didn't get how `ARRAY_SIZE`is related to this PR
     2. >Syntax:
         array_remove(array)
         
        is it possible to invoke `array_remove` for one arg? What is the expected output?
      3. >SELECT array_remove(array(1, 2, 3, null, 3), 3); 
          
         it will not work in Flink since it is not Flink syntax


-- 
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: issues-unsubscribe@flink.apache.org

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