[S2GRAPH-158] Bug fix on String type InnerVal encoding to use UTF-8 charsetName.

• Type: Sub-task
• Status: Done
• Priority: Blocker
• Resolution: Done
• Affects Version/s: 0.2.0
• Fix Version/s: 0.2.0
• Labels: bug
• Assignee: Do Yung Yoon
• Reporter: Do Yung Yoon
• Due: 11/Aug/17
• Created: 07/Aug/17 06:55
• Updated: 07/Aug/17 08:01
• Resolved: 07/Aug/17 08:01

Description

Current master's test failed on following environment and I think it is because of usage of getBytes without specifying encoding on InnerVal class.

More specifically, I think String's getBytes without encoding cause different result.

 case s: String =>
        val pbr = new SimplePositionedMutableByteRange(s.getBytes.length + 3)
        val len = OrderedBytes.encodeString(pbr, s, order)
        pbr.getBytes().take(len)

I am suggesting change String type InnerVal's serialize/deserialize to use UTF-8 encoding so running environment do not break test cases.

Issue Links

• links to: GitHub Pull Request #117