Github user shivzone commented on a diff in the pull request:
https://github.com/apache/incubator-hawq/pull/10#discussion_r41672515
--- Diff:
pxf/pxf-service/src/main/java/com/pivotal/pxf/service/utilities/Utilities.java
---
@@ -13,44 +13,46 @@
*/
public class Utilities {
private static final Log LOG = LogFactory.getLog(Utilities.class);
-
+
/**
* Creates an object using the class name.
* The class name has to be a class located in the webapp's CLASSPATH.
- *
+ *
* @param confClass the class of the metaData used to initialize the
instance
* @param className a class name to be initialized.
* @param metaData input data used to initialize the class
* @return Initialized instance of given className
- * @throws Exception
+ * @throws Exception throws exception if classname was not found in
classpath,
+ * didn't have expected constructor or failed to be
instantiated
*/
- public static Object createAnyInstance(Class<?> confClass, String
className, InputData metaData) throws Exception {
+ public static Object createAnyInstance(Class<?> confClass, String
className, InputData metaData) throws Exception {
--- End diff --
extra space before {
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
