You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Andreas W (JIRA)" <ji...@apache.org> on 2012/04/23 09:22:39 UTC
[jira] [Created] (SOLR-3395) FieldStreamDataSource should handle
null fields
Andreas W created SOLR-3395:
-------------------------------
Summary: FieldStreamDataSource should handle null fields
Key: SOLR-3395
URL: https://issues.apache.org/jira/browse/SOLR-3395
Project: Solr
Issue Type: Improvement
Components: contrib - DataImportHandler
Affects Versions: 3.5
Reporter: Andreas W
Priority: Minor
The {{FieldStreamDataSource}} currently throws a {{DataImportHandlerException}} if a field value is null.
IMHO this is not appropriate: It is legal for field values to be null (like no {{Blob}} exists in a particular row).
I suggest to return an empty InputStream rather than throwing a {{DataImportHandlerException}}.
{code}
public InputStream getData(String query) {
Object o = wrapper.getVariableResolver().resolve(dataField);
if (o == null) {
//better: return new ByteArrayInputStream(new byte[0]);
throw new DataImportHandlerException(SEVERE, "No field available for name : " + dataField);
}
if (o instanceof Blob) {
Blob blob = (Blob) o;
try {
//Most of the JDBC drivers have getBinaryStream defined as public
// so let us just check it
Method m = blob.getClass().getDeclaredMethod("getBinaryStream");
if (Modifier.isPublic(m.getModifiers())) {
return (InputStream) m.invoke(blob);
} else {
// force invoke
m.setAccessible(true);
return (InputStream) m.invoke(blob);
}
} catch (Exception e) {
LOG.info("Unable to get data from BLOB");
return null;
}
} else if (o instanceof byte[]) {
byte[] bytes = (byte[]) o;
return new ByteArrayInputStream(bytes);
} else {
throw new RuntimeException("unsupported type : " + o.getClass());
}
}
{code}
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org
[jira] [Updated] (SOLR-3395) FieldStreamDataSource should handle
null fields
Posted by "Andreas W (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/jira/browse/SOLR-3395?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Andreas W updated SOLR-3395:
----------------------------
Description:
The {{FieldStreamDataSource}} currently throws a {{DataImportHandlerException}} if a field value is null.
IMHO this is not appropriate: It is legal for field values to be null (like no {{Blob}} exists in a particular row).
I suggest to return an empty InputStream rather than throwing a {{DataImportHandlerException}}.
{code}
public InputStream getData(String query) {
Object o = wrapper.getVariableResolver().resolve(dataField);
if (o == null) {
//better: return new ByteArrayInputStream(new byte[0]);
throw new DataImportHandlerException(SEVERE, "No field available for name : " + dataField);
}
if (o instanceof Blob) {
Blob blob = (Blob) o;
try {
//Most of the JDBC drivers have getBinaryStream defined as public
// so let us just check it
Method m = blob.getClass().getDeclaredMethod("getBinaryStream");
if (Modifier.isPublic(m.getModifiers())) {
return (InputStream) m.invoke(blob);
} else {
// force invoke
m.setAccessible(true);
return (InputStream) m.invoke(blob);
}
} catch (Exception e) {
LOG.info("Unable to get data from BLOB");
return null;
}
} else if (o instanceof byte[]) {
byte[] bytes = (byte[]) o;
return new ByteArrayInputStream(bytes);
} else {
throw new RuntimeException("unsupported type : " + o.getClass());
}
}
{code}
Maybe the best implementation would be to distinguish between a none-existing field and a null or empty field.
was:
The {{FieldStreamDataSource}} currently throws a {{DataImportHandlerException}} if a field value is null.
IMHO this is not appropriate: It is legal for field values to be null (like no {{Blob}} exists in a particular row).
I suggest to return an empty InputStream rather than throwing a {{DataImportHandlerException}}.
{code}
public InputStream getData(String query) {
Object o = wrapper.getVariableResolver().resolve(dataField);
if (o == null) {
//better: return new ByteArrayInputStream(new byte[0]);
throw new DataImportHandlerException(SEVERE, "No field available for name : " + dataField);
}
if (o instanceof Blob) {
Blob blob = (Blob) o;
try {
//Most of the JDBC drivers have getBinaryStream defined as public
// so let us just check it
Method m = blob.getClass().getDeclaredMethod("getBinaryStream");
if (Modifier.isPublic(m.getModifiers())) {
return (InputStream) m.invoke(blob);
} else {
// force invoke
m.setAccessible(true);
return (InputStream) m.invoke(blob);
}
} catch (Exception e) {
LOG.info("Unable to get data from BLOB");
return null;
}
} else if (o instanceof byte[]) {
byte[] bytes = (byte[]) o;
return new ByteArrayInputStream(bytes);
} else {
throw new RuntimeException("unsupported type : " + o.getClass());
}
}
{code}
> FieldStreamDataSource should handle null fields
> -----------------------------------------------
>
> Key: SOLR-3395
> URL: https://issues.apache.org/jira/browse/SOLR-3395
> Project: Solr
> Issue Type: Improvement
> Components: contrib - DataImportHandler
> Affects Versions: 3.5
> Reporter: Andreas W
> Priority: Minor
>
> The {{FieldStreamDataSource}} currently throws a {{DataImportHandlerException}} if a field value is null.
> IMHO this is not appropriate: It is legal for field values to be null (like no {{Blob}} exists in a particular row).
> I suggest to return an empty InputStream rather than throwing a {{DataImportHandlerException}}.
> {code}
> public InputStream getData(String query) {
> Object o = wrapper.getVariableResolver().resolve(dataField);
> if (o == null) {
> //better: return new ByteArrayInputStream(new byte[0]);
> throw new DataImportHandlerException(SEVERE, "No field available for name : " + dataField);
> }
> if (o instanceof Blob) {
> Blob blob = (Blob) o;
> try {
> //Most of the JDBC drivers have getBinaryStream defined as public
> // so let us just check it
> Method m = blob.getClass().getDeclaredMethod("getBinaryStream");
> if (Modifier.isPublic(m.getModifiers())) {
> return (InputStream) m.invoke(blob);
> } else {
> // force invoke
> m.setAccessible(true);
> return (InputStream) m.invoke(blob);
> }
> } catch (Exception e) {
> LOG.info("Unable to get data from BLOB");
> return null;
> }
> } else if (o instanceof byte[]) {
> byte[] bytes = (byte[]) o;
> return new ByteArrayInputStream(bytes);
> } else {
> throw new RuntimeException("unsupported type : " + o.getClass());
> }
> }
> {code}
> Maybe the best implementation would be to distinguish between a none-existing field and a null or empty field.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org