You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@madlib.apache.org by GitBox <gi...@apache.org> on 2021/02/18 00:23:15 UTC

[GitHub] [madlib] kaknikhil opened a new pull request #546: DL: Update ind/dep var shape to INTEGER instead of smallint

kaknikhil opened a new pull request #546:
URL: https://github.com/apache/madlib/pull/546


   JIRA: MADLIB-1468
   
   training_preprocessor_dl() function fails with the following error when the
   number of images in a buffer is > range for smallint i.e. -32768 to +32767.
   ```
   ERROR: spiexceptions.NumericValueOutOfRange: smallint out of range
   ```
   This commit replaces smallint with INTEGER for the shape column.
   
   Co-authored-by: Nikhil Kak<nk...@vmware.com>
   
   <!--  
   
   Thanks for sending a pull request!  Here are some tips for you:
   1. Refer to this link for contribution guidelines https://cwiki.apache.org/confluence/display/MADLIB/Contribution+Guidelines
   2. Please Provide the Module Name, a JIRA Number and a short description about your changes.
   -->
   
   - [ ] Add the module name, JIRA# to PR/commit and description.
   - [ ] Add tests for the change. 
   
   


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [madlib] fmcquillan99 commented on pull request #546: DL: Update ind/dep var shape to INTEGER instead of smallint

Posted by GitBox <gi...@apache.org>.
fmcquillan99 commented on pull request #546:
URL: https://github.com/apache/madlib/pull/546#issuecomment-781538750


   ready to merge


----------------------------------------------------------------
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:
users@infra.apache.org



[GitHub] [madlib] khannaekta merged pull request #546: DL: Update ind/dep var shape to INTEGER instead of smallint

Posted by GitBox <gi...@apache.org>.
khannaekta merged pull request #546:
URL: https://github.com/apache/madlib/pull/546


   


----------------------------------------------------------------
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:
users@infra.apache.org