[HDFS-3916] libwebhdfs (C client) code cleanups

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.2-alpha
• Fix Version/s: 2.0.3-alpha
• Component/s: webhdfs
• Hadoop Flags: Reviewed
• Assignee: Colin McCabe
• Reporter: Colin McCabe
• Created: 11/Sep/12 01:27
• Updated: 15/Feb/13 13:12
• Resolved: 04/Oct/12 03:04

Description

Code cleanups in libwebhdfs.

• don't duplicate exception.c, exception.h, expect.h, jni_helper.c. We have one copy of these files; we don't need 2.
• remember to set errno in all public library functions (this is part of the API)
• fix undefined symbols (if a function is not implemented, it should return ENOTSUP, but still exist)
• don't expose private data structures in the (end-user visible) public headers
• can't re-use hdfsBuilder as hdfsFS, because the strings in hdfsBuilder are not dynamically allocated.

Issue Links

• relates to: HDFS-3952 Support hdfsHFlush and hdfsFlush in libwebhdfs (C client)
• relates to: HDFS-2656 Implement a pure c client based on webhdfs

Sub-Tasks

• libwebdhfs code cleanup: string processing and using strerror consistently to handle all errors: Closed
• libwebhdfs testing code cleanup: Closed
• Fix compile warnings for libwebhdfs: Open