Github user jaanai0 commented on a diff in the pull request:
https://github.com/apache/phoenix/pull/355#discussion_r224666108
--- Diff:
phoenix-core/src/main/java/org/apache/phoenix/schema/KeyValueSchema.java ---
@@ -121,7 +122,15 @@ public boolean isNull(int position, ValueBitSet
bitSet) {
int nBytes = ptr.getLength();
b = ensureSize(b, offset, offset + nBytes);
System.arraycopy(ptr.get(), ptr.getOffset(), b,
offset, nBytes);
- offset += nBytes;
+ if (field.getMaxLength() != null &&
field.getMaxLength() > 0 ) {
--- End diff --
@twdsilva Yes, we should only pad Pchar type, through other data types of
fixed width which excepts for PChar have a permanent length of columns. But in
this cases that cast type of fixed columns, the results may be incorrect.
---
