You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2022/01/15 23:10:38 UTC

[GitHub] [drill] cgivre commented on pull request #2427: DRILL-8095: upgrade to poi 5.2.0

cgivre commented on pull request #2427:
URL: https://github.com/apache/drill/pull/2427#issuecomment-1013768959


   > @cgivre see latest comment on https://issues.apache.org/jira/browse/DRILL-8095 - not loading the styles breaks existing Drill tests - so I abandoned that bit of the change
   
   Roger that.... Sorry I didn't see that.  We don't have to include it in this PR, but if you can recommend a better way of inferring the data type that would be more performant, I'm all ears.  I did it using the Cell style because that seemed to make the most sense.   Thanks!


-- 
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: dev-unsubscribe@drill.apache.org

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