jpisaac commented on a change in pull request #442: PHOENIX-5122: PHOENIX-4322
breaks client backward compatibility
URL: https://github.com/apache/phoenix/pull/442#discussion_r259969994
##########
File path:
phoenix-core/src/main/java/org/apache/phoenix/expression/RowValueConstructorExpression.java
##########
@@ -46,14 +47,30 @@
private ImmutableBytesWritable literalExprPtr;
private int partialEvalIndex = -1;
private int estimatedByteSize;
+
+ // The boolean field that indicated the object is a literal constant,
+ // has been repurposed to a bitset and now holds additional information.
+ // This is to facilitate b/w compat to 4.13 clients.
+ // @see <a
href="https://issues.apache.org/jira/browse/PHOENIX-5122";>PHOENIX-5122</a>
+ private BitSet extraFields;
+
+ private enum ExtraFieldPosition {
Review comment:
@dbwong @ChinmaySKulkarni
Let me know if we need more testing than -
org.apache.phoenix.end2end.InListIT
org.apache.phoenix.end2end.SortOrderIT
org.apache.phoenix.end2end.RowValueConstructorIT
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
