Issue in Excel Output while using SXSSF Transformer with JXLS (>=2.7.0)

[vaibhavk311]

Hey Team,

I've observed that the static cells are getting erased in output when I'm using SXSSF transformer to transform and apply the area as per the comments from template.

I'm using JXLS 2.11.0 and JXLS-POI 2.11.0.

Moreover I've noted that such behavior is observed in versions newer that JXLS 2.7.0 with SXSSF Transformer

This behavior exists with fixed set of pattern i.e. in which static cell is either to left or right of iterated cell.

Sample Template

Output : JXLS 2.11.0 Output

Note that A3 cell("Manager") from Template is erased.

Any comments/lead/resolution if bug would be highly appreciated

[SoltauFintel]

in general: You can not use Streaming for everything. Usually it's used for 1 simple data list on 1 sheet.

So it works before 2.7.0 (what version have you tested?) and does not work with 2.11.0 ? (Maybe caused by PR #70 ..?)

[vaibhavk311]

@SoltauFintel It works fine for JXLS 2.6.x version but it is breaking for versions greater that (>=2.7.0).
Can you help us out here in case we want to preserve the same functionality offered in 2.7.x but for 2.11.0 JXLS-Version ?

[SoltauFintel]

If you want to speed things up you can push a PR containing a (failin') testcasse. Or maybe in the 2nd step search for the bug.

[SoltauFintel]

I cannot reproduce this. In my opinion you cannot use streaming for this template.

We need a testcase; pushed as PR. Please create the template from scratch using only Microsoft Excel.

[vaibhavk311]

@SoltauFintel
Can you please give me the access to Push the PR(Code) to a remote branch.
My githubId is vaibhavk311
Currently it says that :
remote: Permission to jxlsteam/jxls.git denied to vaibhavk311.
fatal: unable to access 'https://github.com/jxlsteam/jxls.git/': The requested URL returned error: 403

[SoltauFintel]

@vaibhavk311
You must fork this repo. see https://github.com/jxlsteam/jxls/blob/master/CONTRIBUTING.md

[vaibhavk311]

Sure thanks @SoltauFintel
PR : #155

[SoltauFintel]

@vaibhavk311
Did you recreate the template from scratch using MS Excel? Sometimes templates are just broken.

[vaibhavk311]

@SoltauFintel yes i recreated it from scratch

[SoltauFintel]

Analysis

• cause is not PR #68 clearCellValue workaround for every setCellFormula() call #70
• cause is not POI 4.1.0 (same error with 4.0.1)
• yes, works with tag v2.6.0
• yes, works not with tag 2.7.0
• cause is commit 5354bea
  issue 160 - add isForwardOnly() method to Transformer and processing static cells differently depending on this flag
  issue#160 jx:each with direction=RIGHT with SXSSF Transformer rewrites static cells
  (we cannot read the old issue at the moment...)
  First released with 2.7.0

[vaibhavk311]

@SoltauFintel Thanks for RCA :)
What is the resolution to be made

[SoltauFintel]

What means RCA?

[SoltauFintel]

@vaibhavk311
Let's wait for leonate.
You could try to set transformer.forwardOnly to false.

[vaibhavk311]

@SoltauFintel
RCA-> Root Cause Analysis
Sure i'll try doing this but transformer.forwardOnly will not support streaming right ?

[leonate]

AFAIR isForwardOnly property was introduced to differentiate between streaming and not streaming transformers
public boolean isForwardOnly() { return isStreaming(); }

With a streaming transformer (SXSSF) we cannot rewrite already written cells.
But I suppose some regression was introduced with the change. The original bug link is https://bitbucket.org/leonate/jxls/issues/160/jx-each-with-direction-right-with-sxssf .

[vaibhavk311]

@leonate
Thanks for the response :)
What is the resolution to be made

[ankurkumarHL]

@vaibhavk311 I am facing same issue in, did you get the solution or work around?

[ankurkumarHL]

@leonate when can we expect this regression to be fixed?

[SoltauFintel]

There's no solution yet. See comment in PR 156. JXLS is no full-time project. It's done when it's done. :-) And there is no release date yet.

[leonate]

The issue fixed in pull request #159

[vaibhavk311]

Thanks @leonate , @SoltauFintel :)
Just one Quick Question :
Will it be released in JXLS - 2.12.0 Version ?

[leonate]

Will it be released in JXLS - 2.12.0 Version ?

That's the plan.

[SoltauFintel]

This change causes error #173