Manno15 commented on a change in pull request #2255:
URL: https://github.com/apache/accumulo/pull/2255#discussion_r705453302
##########
File path: test/src/main/java/org/apache/accumulo/test/ShellServerIT.java
##########
@@ -1939,6 +1939,19 @@ public void scans() throws Exception {
ts.exec("deletetable -f t");
}
+ @Test
+ public void scansWithColon() throws Exception {
+ ts.exec("createtable twithcolontest");
+ make10WithColon();
+ ts.exec("scan -r row0 -c c\\:f:col0", true, "value");
+ String result = ts.exec("scan -b row1 -c c\\:f:col1 -e row1");
+ assertEquals(2, result.split("\n").length);
+ result = ts.exec("scan -r row0 -c c:f");
+ assertEquals(1, result.split("\n").length);
Review comment:
With the latest changes, it seems the use of an '\\' character is
unnecessary now. The reason your second check doesn't pass is because it does
not contain the :col1 (which isn't a requirement for scan, just a quirk of the
changes you added) at the end.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
