[CALCITE-1752] Use URLDecoder#decode to handle URLs acquired from getResource(..) calls in test cases

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: avatica-1.10.0, 1.13.0
• Assignee: Josh Elser
• Reporter: Josh Elser
• Created: 11/Apr/17 20:42
• Updated: 30/May/17 16:23
• Resolved: 25/Apr/17 20:11

Description

From https://github.com/apache/calcite/commit/6543c0fd8b63efb00d035790209f8546668d7aa5#commitcomment-21723380:

michaelmior suggested that it would be better to replace my String.replace("%20", " ") with java.net.URLDecoder.decode instead. I am apt to agree with him

Maybe we can pull this into a utility instead of littering it around test cases in avatica and calcite.

Issue Links

• links to: GitHub Pull Request #7