https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
Javen O'Neal changed:
What|Removed |Added
Keywords||PatchAvailable
Status|NEW
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #2 from Renjith R ---
Created attachment 34757
--> https://bz.apache.org/bugzilla/attachment.cgi?id=34757&action=edit
Patch which implements the interfaces Cell, Row, Sheet & Workbook
I have implemented the Cell, Row, Sheet & Wor
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
Renjith R changed:
What|Removed |Added
Status|NEEDINFO|NEW
--- Comment #3 from Renjith R ---
I h
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
Dominik Stadler changed:
What|Removed |Added
Severity|normal |enhancement
--
You are receiving th
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #4 from Renjith R ---
Gentle reminder. Did someone got a chance to look into?
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsub
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #5 from Renjith R ---
I'll continue to add the functionalities and post the patches here. I will be
glad if someone got a chance to review my work. Let me know if you want me to
look at anything specific or please guide me if I am m
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
Dominik Stadler changed:
What|Removed |Added
Status|NEW |NEEDINFO
--- Comment #6 from Dominik
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #7 from Renjith R ---
Thanks a lot for the Inputs, Dominik. Let me work out on these points and I'll
get back to you.
--
You are receiving this mail because:
You are the assignee for the bug.
--
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #8 from Renjith R ---
Created attachment 34943
--> https://bz.apache.org/bugzilla/attachment.cgi?id=34943&action=edit
Version 3: Patch with some of the review comments implemented
Hi..
I have tried to implement most of the comme
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #9 from Javen O'Neal ---
Could you also add a @NotImplemented annotation to methods where you throw
Unsupported operation exception? This will make it clear to the Javadocs
readers that the method isn't implemented yet.
https://poi
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #10 from Renjith R ---
Created attachment 34945
--> https://bz.apache.org/bugzilla/attachment.cgi?id=34945&action=edit
Version 4: Added @NotImplemented. Added feature getColumnIndex
Thanks a lot for the suggestion.
Added @NotImpl
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #11 from Renjith R ---
I was analyzing 'DateFormatConverter' to see if I can reuse any logic from it.
But, as per my understanding, this class is more helpful while writing data to
excel. eg convert(Locale.JAPANESE, "dd , ")
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #12 from Renjith R ---
I was analyzing 'DateFormatConverter' to see if I can reuse any logic from it.
But, as per my understanding, this class is more helpful while writing data to
excel. eg convert(Locale.JAPANESE, "dd , ")
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #13 from Renjith R ---
Did someone get a chance to review the code?
I'd be glad to see some comments.
--
You are receiving this mail because:
You are the assignee for the bug.
-
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #14 from PJ Fanning ---
I added a StaxHelper class today because I think it is a good idea for us to
apply default configuration to the factories. One benefit is to protect against
XML Entity Expansion attacks.
Would you be able to
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #15 from Renjith R ---
Sure, I'll look into it.
Hope (https://bz.apache.org/bugzilla/show_bug.cgi?id=61213) is the one you are
talking about.
--
You are receiving this mail because:
You are the assignee for the bug.
-
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #16 from PJ Fanning ---
Renjith - the StaxHelper is already merged to the svn trunk (src/java). The
rest of the change for using StAX parser for issue 61213 is still under
discussion.
--
You are receiving this mail because:
You ar
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
Dominik Stadler changed:
What|Removed |Added
Depends on||61213
Referenced Bugs:
https://bz.
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #17 from PJ Fanning ---
Renjith - could you attach SpreadSheetSample04022017.xlsx to the issue?
--
You are receiving this mail because:
You are the assignee for the bug.
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #18 from Renjith R ---
You can take it from git.
https://github.com/ranju4u6/poi/blob/Startup/test-data/spreadsheet/SpreadSheetSample04022017.xlsx
Sorry, I did not get a chance to look into your code due to tight work
schedule.
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
Bug 60707 depends on bug 61213, which changed state.
Bug 61213 Summary: Replace SXSSFWorkbook copyStreamAndInjectWorksheet with StAX
equivalent
https://bz.apache.org/bugzilla/show_bug.cgi?id=61213
What|Removed
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #19 from PJ Fanning ---
I created a new branch and Pull Request for the work to date.
I took Renjith's patch and did a little tidy up.
https://github.com/apache/poi/pull/64
I think we need a fair amount of extra coverage (using dif
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #24 from PJ Fanning ---
this is abandoned
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@poi.
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
PJ Fanning changed:
What|Removed |Added
Status|NEEDINFO|RESOLVED
Resolution|---
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
Dominik Stadler changed:
What|Removed |Added
Summary|[PATCH] Reading very large |[PATCH] Reading very large
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #20 from Renjith R ---
Hi PJ Fanning, I believe you have done some code review changes and some tidy
ups.Thanks for that. So am I good to take the latest code from your
repository(https://github.com/pjfanning/poi.git) with Branch:
"
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #21 from PJ Fanning ---
Renjith - feel free to fork my branch.
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail:
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #22 from PJ Fanning ---
One alternative worth trying is
https://github.com/monitorjbl/excel-streaming-reader which is built on top of
Apache POI.
--
You are receiving this mail because:
You are the assignee for the bug.
--
https://bz.apache.org/bugzilla/show_bug.cgi?id=60707
--- Comment #23 from Renjith R ---
Cool! Looks promising.
--
You are receiving this mail because:
You are the assignee for the bug.
-
To unsubscribe, e-mail: dev-unsubscr...@
29 matches
Mail list logo
