[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-19 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16620300#comment-16620300 ] Robert Munteanu commented on SLING-7890: Confirmed, thanks a lot [~jebailey]! > JSON-related IT

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-18 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16619481#comment-16619481 ] Jason E Bailey commented on SLING-7890: --- [~rombert] You should be able to update the tests and have

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-18 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16619174#comment-16619174 ] Jason E Bailey commented on SLING-7890: --- looking into it > JSON-related IT failures when upgrading

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-18 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16619159#comment-16619159 ] Robert Munteanu commented on SLING-7890: [~jebailey] - I still get test failures with: * locally

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-18 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16618960#comment-16618960 ] Robert Munteanu commented on SLING-7890: [~jebailey] - I'll release the IT bundle anyway for the

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-18 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16618899#comment-16618899 ] Jason E Bailey commented on SLING-7890: --- [~rombert] Yes it's ready to be released. I was hoping to

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-18 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16618687#comment-16618687 ] Robert Munteanu commented on SLING-7890: [~jebailey] - so should we be safe to release the new

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-17 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617816#comment-16617816 ] Jason E Bailey commented on SLING-7890: --- [~rombert] 2.1.34, as it stands, has a bug and should not

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-17 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16617787#comment-16617787 ] Robert Munteanu commented on SLING-7890: [~jebailey] - are you still looking into this? It's ok

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-11 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16611273#comment-16611273 ] Bertrand Delacretaz commented on SLING-7890: Ok got it, so I agree with changing the

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-11 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16610943#comment-16610943 ] Jason E Bailey commented on SLING-7890: --- [~bdelacretaz] You got the tests right, it's that I had

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-11 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16610869#comment-16610869 ] Bertrand Delacretaz commented on SLING-7890: I have added some tests for the {{JsonRenderer}}

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-11 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16610724#comment-16610724 ] Robert Munteanu commented on SLING-7890: Then I guess that the correct behaviour is to have the

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-11 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16610720#comment-16610720 ] Jason E Bailey commented on SLING-7890: --- [~rombert] Correct. > JSON-related IT failures when

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-11 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16610718#comment-16610718 ] Robert Munteanu commented on SLING-7890: Nice investigation [~jebailey]! So, IIUC, the "native"

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-11 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16610625#comment-16610625 ] Jason E Bailey commented on SLING-7890: --- Additional information The issue that is occurring is

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-11 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16610242#comment-16610242 ] Robert Munteanu commented on SLING-7890: {quote}Personally I would prefer that we keep the old,

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-10 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16609975#comment-16609975 ] Jason E Bailey commented on SLING-7890: --- [~bdelacretaz] So something is odd here. The reason this

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-10 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16609770#comment-16609770 ] Bertrand Delacretaz commented on SLING-7890: Ok, got it - sorry that I didn't figure it out

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-10 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16609761#comment-16609761 ] Jason E Bailey commented on SLING-7890: --- [~bdelacretaz] yes there is a difference in the output,

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-10 Thread Bertrand Delacretaz (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16609733#comment-16609733 ] Bertrand Delacretaz commented on SLING-7890: The commit that you mention is

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-07 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607251#comment-16607251 ] Jason E Bailey commented on SLING-7890: --- [~rombert] I've committed changes to both the default

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-07 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607120#comment-16607120 ] Jason E Bailey commented on SLING-7890: --- There's a number of tests failing related to my changes

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-07 Thread Jason E Bailey (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16607027#comment-16607027 ] Jason E Bailey commented on SLING-7890: --- I see the problem. I'll work on the fix and a unit test.

[jira] [Commented] (SLING-7890) JSON-related IT failures when upgrading sling.servlets to 2.1.34

2018-09-07 Thread Robert Munteanu (JIRA)
[ https://issues.apache.org/jira/browse/SLING-7890?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16606879#comment-16606879 ] Robert Munteanu commented on SLING-7890: [~jebailey] - since you did most of the changes between