[GitHub] [netbeans] matthiasblaesing edited a comment on pull request #2169: [NETBEANS-4415] Export dataview results to file

[GitBox <gi...@apache.org>]

matthiasblaesing edited a comment on pull request #2169:
URL: https://github.com/apache/netbeans/pull/2169#issuecomment-657262031


   Thank you for the research.
   
   > Having said all that I am in favor of providing the csv/tsv option and additionally the .xlsx option by using **fastexcel**. Old .xls format was coming for free when using POI and code-wise it was pretty similar to .xlsx, so it made sense to export in that too. Now that we are talking about using two different libraries I think we can skip .xls support because the format is too old, the produced files are larger than .xlsx in general and I don't think adding it adds any actual value.
   
   I think useing fastexcel is a sensible approach. If I'm not mistaken the footprint will shrink to ~100kB. (opczip + fastexcel). That sounds good to me.
   
   Edit: One more relevant thing: The license situation with fastexcel is good (both dependencies are ALv2, LGPL is a no-go for ASF projects).


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



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists