abdullah alamoudi has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/2090

Change subject: [NO ISSUE][RT] Prevent setting AMutableString value to null
......................................................................

[NO ISSUE][RT] Prevent setting AMutableString value to null

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
- An instance of AMutableString can't hold null value.

Change-Id: I8e7b19b190120e64ec78b41f26ef54fce9660e27
---
M 
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/base/AMutableString.java
1 file changed, 5 insertions(+), 0 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/90/2090/1

diff --git 
a/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/base/AMutableString.java
 
b/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/base/AMutableString.java
index 33217cd..b2a4a5e 100644
--- 
a/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/base/AMutableString.java
+++ 
b/asterixdb/asterix-om/src/main/java/org/apache/asterix/om/base/AMutableString.java
@@ -20,11 +20,16 @@
 
 public class AMutableString extends AString {
 
+    private static final long serialVersionUID = 1L;
+
     public AMutableString(String value) {
         super(value);
     }
 
     public void setValue(String value) {
+        if (value == null) {
+            throw new NullPointerException();
+        }
         this.value = value;
     }
 }

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/2090
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8e7b19b190120e64ec78b41f26ef54fce9660e27
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <[email protected]>

Reply via email to