Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-2103][STO] Too many disk components for
CorrelatedPolicy
..
Patch Set 17: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/2018
To
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-2103][STO] Too many disk components for
CorrelatedPolicy
..
Patch Set 16:
@Ian, does the cb-jenkins matters?
--
To view, visit https://asterix-gerri
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-2103][STO] Too many disk components for
CorrelatedPolicy
..
Patch Set 16: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/2018
To
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-2119][COMP] Fix variable ordering of project pushdown
..
Patch Set 3: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/2048
To unsub
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-2119][COMP] Fix variable ordering of project pushdown
..
Patch Set 3:
(1 comment)
https://asterix-gerrit.ics.uci.edu/#/c/2048/3/hyracks-fullstack/alge
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-2103][STO] Too many disk components for
CorrelatedPolicy
..
Patch Set 8:
(4 comments)
I left some minor comments.
https://asterix-gerrit.ics.uci.edu
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-2103][STO] Too many disk components for
CorrelatedPolicy
..
Patch Set 3:
@Luo any ideas about the "cb-jenkins" test fail? I think you can ask @Adbulla
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-2103][STO] Too many disk components for
CorrelatedPolicy
..
Patch Set 2:
(2 comments)
I have two minor comments.
https://asterix-gerrit.ics.uci.edu/
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-1946][STO][IDX]Create RTree/InvertedIdx for
Correlated Datasets
..
Patch Set 6:
Cloudberry currently only need an inverted index. So I didn't test oth
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-1946][STO][IDX]Create RTree/InvertedIdx for
Correlated Datasets
..
Patch Set 6:
@Chenluo, not much about this patch on the code side. I want to updat
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-1946][STO][IDX]Create RTree/InvertedIdx for
Correlated Datasets
..
Patch Set 5: Code-Review+1
Maybe Abdullah or Yingyi can have a pass finally?
--
T
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-1952][TX][IDX]Log incoming filter vals
..
Patch Set 10: Code-Review+1
I see. I will give a +1. I think the upsert issue as Murtadha's pointed out
coul
Jianfeng Jia has posted comments on this change.
Change subject: [ASTERIXDB-1952][TX][IDX]Log incoming filter vals
..
Patch Set 10:
(1 comment)
why is shows can't merge?
https://asterix-gerrit.ics.uci.edu/#/c/1798/5/hyracks-f
Jianfeng Jia has posted comments on this change.
Change subject: Avoid always merging old components in prefix policy
..
Patch Set 5:
(1 comment)
https://asterix-gerrit.ics.uci.edu/#/c/1818/5/hyracks-fullstack/hyracks/hyracks-
Jianfeng Jia has posted comments on this change.
Change subject: Avoid always merging old components in prefix policy
..
Patch Set 4:
(7 comments)
some minor comments.
https://asterix-gerrit.ics.uci.edu/#/c/1818/4/hyracks-ful
Jianfeng Jia has submitted this change and it was merged.
Change subject: Change logical plan to apply filter from 2ndary index
..
Change logical plan to apply filter from 2ndary index
- Changes the IntroduceLSMComponentFilterR
Hello Ian Maxon, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#17).
Change subject: Change logical plan to apply filter from 2ndary index
.
Jianfeng Jia has posted comments on this change.
Change subject: Change logical plan to apply filter from 2ndary index
..
Patch Set 15:
(3 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1727/15/asterixdb/asterix-algebra/src/
Hello Ian Maxon, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#16).
Change subject: Change logical plan to apply filter from 2ndary index
.
Hello Ian Maxon, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#13).
Change subject: Change logical plan to apply filter from 2ndary index
.
Jianfeng Jia has posted comments on this change.
Change subject: Change logical plan to apply filter from 2ndary index
..
Patch Set 12:
(27 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1727/11/asterixdb/asterix-algebra/src
Hello Ian Maxon, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#12).
Change subject: Change logical plan to apply filter from 2ndary index
.
Hello Ian Maxon, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#10).
Change subject: Change logical plan to apply filter from 2ndary index
.
Jianfeng Jia has posted comments on this change.
Change subject: Change logical plan to apply filter from 2ndary index
..
Patch Set 9:
(2 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1727/7/asterixdb/asterix-app/data/twitt
Hello Ian Maxon, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#9).
Change subject: Change logical plan to apply filter from 2ndary index
..
Jianfeng Jia has posted comments on this change.
Change subject: Change logical plan to apply filter from 2ndary index
..
Patch Set 7:
(9 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1727/7/.gitignore
File .gitignore:
> W
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#8).
Change subject: Change logical plan to apply filter from 2ndary index
..
Change lo
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#7).
Change subject: Change logical plan to apply filter from 2ndary index
..
Change lo
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..
Patch Set 4: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To unsubscribe, visit
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..
Patch Set 3: Code-Review+1
LGTM. I think we should also fix that two formatting issues?
--
To view, visit
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1905: Incorrect filter for post-load sidx
..
Patch Set 2:
any reason for the verify fail?
--
To view, visit https://asterix-gerrit.ics.uci.edu/1743
To
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#4).
Change subject: Change logical plan to apply filter from 2ndary index
..
Change lo
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#3).
Change subject: Change logical plan to apply filter from 2ndary index
..
Change lo
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1727
to look at the new patch set (#2).
Change subject: Change logical plan to apply filter from 2ndary index
..
Change lo
Jianfeng Jia has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1727
Change subject: Change logical plan to apply filter from 2ndary index
..
Change logical plan to apply filter from 2ndary index
- Chang
Jianfeng Jia has posted comments on this change.
Change subject: Carry filter in 2ndary-to-primary index search
..
Patch Set 7:
(3 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1720/7/hyracks-fullstack/hyracks/hyracks-examp
Hello Yingyi Bu, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1720
to look at the new patch set (#8).
Change subject: Carry filter in 2ndary-to-primary index search
..
Carry
Jianfeng Jia has posted comments on this change.
Change subject: Carry filter in 2ndary-to-primary index search
..
Patch Set 6:
(20 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1720/6/asterixdb/asterix-algebra/src/main/jav
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1720
to look at the new patch set (#7).
Change subject: Carry filter in 2ndary-to-primary index search
..
Carry filter in
Jianfeng Jia has posted comments on this change.
Change subject: Carry filter in 2ndary-to-primary index search
..
Patch Set 6:
(6 comments)
@Yingyi @Ian, this is patch expose the filter value and pass it along with
index rec
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1720
to look at the new patch set (#6).
Change subject: Carry filter in 2ndary-to-primary index search
..
Carry filter in
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1720
to look at the new patch set (#5).
Change subject: Carry filter in 2ndary-to-primary index search
..
Carry filter in
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1720
to look at the new patch set (#4).
Change subject: Carry filter in 2ndary-to-primary index search
..
Carry filter in
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1720
to look at the new patch set (#3).
Change subject: Carry filter in 2ndary-to-primary index search
..
Carry filter in
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1720
to look at the new patch set (#2).
Change subject: Carry filter in 2ndary-to-primary index search
..
Carry filter in
Jianfeng Jia has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1720
Change subject: Carry filter in 2ndary-to-primary index search
..
Carry filter in 2ndary-to-primary index search
Change-Id: I287f1dbd2
Jianfeng Jia has posted comments on this change.
Change subject: Hyracks Unit tests for LSM Filters
..
Patch Set 10: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/1704
To unsubscribe, visit https://asterix
Jianfeng Jia has posted comments on this change.
Change subject: Hyracks Unit tests for LSM Filters
..
Patch Set 7: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/1704
To unsubscribe, visit https://asterix-
Jianfeng Jia has posted comments on this change.
Change subject: Hyracks Unit tests for LSM Filters
..
Patch Set 3:
(1 comment)
do we have the corresponding RTree/InvertedIndex merge filter test?
https://asterix-gerrit.ics.uc
Jianfeng Jia has posted comments on this change.
Change subject: Document the Query Service API
..
Patch Set 3: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/1698
To unsubscribe, visit https://asterix-gerr
Jianfeng Jia has posted comments on this change.
Change subject: Document the Query Service API
..
Patch Set 1:
(1 comment)
https://asterix-gerrit.ics.uci.edu/#/c/1698/1/asterixdb/asterix-doc/src/site/markdown/api.md
File aste
Jianfeng Jia has posted comments on this change.
Change subject: Fix for ASTERIXDB-1886: Filter not selective
..
Patch Set 2:
Any test case we can add to make filter safer?
--
To view, visit https://asterix-gerrit.ics.uci.edu
Jianfeng Jia has posted comments on this change.
Change subject: Fix for ASTERIXDB-1879
..
Patch Set 7: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1672
To unsubscribe, visit https://asterix-gerrit.ics.u
Jianfeng Jia has posted comments on this change.
Change subject: Fix for ASTERIXDB-1879
..
Patch Set 6:
(2 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1672/6/hyracks-fullstack/hyracks/hyracks-tests/hyracks-storage-am-lsm-
Jianfeng Jia has submitted this change and it was merged.
Change subject: Print filter information in logical plan
..
Print filter information in logical plan
Change-Id: I8c3893fc54337695df4837e34ad7f7e89758718e
Reviewed-on: ht
Jianfeng Jia has posted comments on this change.
Change subject: Print filter information in logical plan
..
Patch Set 2:
(2 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1673/2//COMMIT_MSG
Commit Message:
PS2, Line 9: wor
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1673
to look at the new patch set (#3).
Change subject: Print filter information in logical plan
..
Print filter informati
Jianfeng Jia has posted comments on this change.
Change subject: Fix for ASTERIXDB-1879
..
Patch Set 1:
I'm thinking an easier unit test. E.g., we can write a filter information to a
page and read it later to see if we can get
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1673
to look at the new patch set (#2).
Change subject: Print filter information in logical plan
..
Print filter informati
Jianfeng Jia has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1673
Change subject: Print filter information in logical plan
..
Print filter information in logical plan
The following commits from your w
Jianfeng Jia has posted comments on this change.
Change subject: Fix for ASTERIXDB-1879
..
Patch Set 1:
so this means we have to re-ingest the data since the filter is not write
correctly
--
To view, visit https://aster
Jianfeng Jia has posted comments on this change.
Change subject: Fix for ASTERIXDB-1879
..
Patch Set 1:
should we have some sort of filter related unit test?
--
To view, visit https://asterix-gerrit.ics.uci.edu/1672
To unsubs
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1877: Tokenizer in FullText Search fix
..
Patch Set 4: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1668
To unsubscribe, visit ht
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1877: Tokenizer in FullText Search fix
..
Patch Set 4:
Any plan to merge it soon? Cloudberry need it to make the current demo work :-)
--
To view, vis
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1327, ASTERIXDB-1362 Fixed circle-point intersect
function
..
Patch Set 3: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1558
To
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function
..
Patch Set 8: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1481
To unsubscribe,
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function
..
Patch Set 6:
Looks good. But I still feel there should be some simple *JUnit* test for the
edit dist
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function
..
Patch Set 4:
Oh, one more thing, do you have some test cases that actually trigger the early
termina
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1778: Optimize the edit-distance-check function
..
Patch Set 4:
(10 comments)
Just some minor comments.
https://asterix-gerrit.ics.uci.edu/#/c/1481/4/
Jianfeng Jia has submitted this change and it was merged.
Change subject: Support LSM-filter on multiple input paths
..
Support LSM-filter on multiple input paths
The IntroduceLSMComponentFilterRule only checked one input path,
Jianfeng Jia has posted comments on this change.
Change subject: Support LSM-filter on multiple input paths
..
Patch Set 3: -Code-Review
--
To view, visit https://asterix-gerrit.ics.uci.edu/1356
To unsubscribe, visit https://a
Hello Taewoo Kim, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1356
to look at the new patch set (#3).
Change subject: Support LSM-filter on multiple input paths
..
Support
Jianfeng Jia has posted comments on this change.
Change subject: Support LSM-filter on multiple input paths
..
Patch Set 3: Code-Review+2
Forward Taewoo's +2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1356
To unsubs
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/1356
to look at the new patch set (#2).
Change subject: Support filter on multiple input path
..
Support filter on multipl
Jianfeng Jia has abandoned this change.
Change subject: Not suppose to MERGE, for discussion purpose The following
commits from your working branch will be included:
..
Abandoned
--
To view, visit https://asterix-gerrit.ics.u
Jianfeng Jia has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1356
Change subject: Support filter on multiple input path
..
Support filter on multiple input path
The following commits from your working
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1700: fixed multiple same type of index application
error on the same field
..
Patch Set 3: Code-Review+2
--
To view, visit https://asterix-gerrit.ics
Jianfeng Jia has posted comments on this change.
Change subject: Make Lock Manager Statistics at FINE granularity
..
Patch Set 4: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1303
To unsubscribe, visit ht
Jianfeng Jia has posted comments on this change.
Change subject: Make Lock Manager Statistics at FINE granularity
..
Patch Set 3: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/1303
To unsubscribe, visit ht
Jianfeng Jia has posted comments on this change.
Change subject: Make Lock Manager Statistics at FINE granularity
..
Patch Set 3:
(1 comment)
https://asterix-gerrit.ics.uci.edu/#/c/1303/3/asterixdb/asterix-events/src/main/reso
Jianfeng Jia has posted comments on this change.
Change subject: Not suppose to MERGE, for discussion purpose The following
commits from your working branch will be included:
..
Patch Set 1:
That `println` only print if anythi
Jianfeng Jia has posted comments on this change.
Change subject: Tests for ASTERIXDB-1636
..
Patch Set 4:
I have one small question that maybe not relevant to this patch.
If you look at the *.aql, it is often ordered alphabeti
Jianfeng Jia has posted comments on this change.
Change subject: Index-only plan step 2: Added SplitOperator
..
Patch Set 12: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1196
To unsubscribe, visit https:
Jianfeng Jia has posted comments on this change.
Change subject: Index-only plan step 2: Added SplitOperator
..
Patch Set 11:
(1 comment)
Just one comment
https://asterix-gerrit.ics.uci.edu/#/c/1196/11/hyracks-fullstack/algeb
Jianfeng Jia has posted comments on this change.
Change subject: Index-only plan step 2: Added SplitOperator
..
Patch Set 10:
Like this kind of test?
:https://github.com/apache/asterixdb/blob/bce00de839d844e41de2958e1494a6
Jianfeng Jia has posted comments on this change.
Change subject: Index-only plan step 2: Added SplitOperator
..
Patch Set 10:
(6 comments)
Looks good in general. I have two main comments:
1. I saw this "numberOfNonMaterialize
Jianfeng Jia has posted comments on this change.
Change subject: Updated DeletableFrameTupleAppender to support reusing index
slots.
..
Patch Set 2:
(2 comments)
https://asterix-gerrit.ics.uci.edu/#/c/1214/2/hyracks-fullstack
Jianfeng Jia has posted comments on this change.
Change subject: Fix for ASTERIXDB-1636
..
Patch Set 4: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/1205
To unsubscribe, visit https://asterix-gerrit.ics.u
Jianfeng Jia has posted comments on this change.
Change subject: Updated DeletableFrameTupleAppender to support reusing index
slots.
..
Patch Set 1:
(9 comments)
Looks good in general. Just a few minor comments.
https://aste
Jianfeng Jia has posted comments on this change.
Change subject: Fix for ASTERIXDB-1636
..
Patch Set 4:
(1 comment)
I think maybe Taewoo knows better about the changes.
Do we have some sort of unit test for storage codes? thi
Jianfeng Jia has posted comments on this change.
Change subject: Adding a new 101 tutorial for SQL++.
..
Patch Set 1:
Thanks Yingyi for the explaination!
--
To view, visit https://asterix-gerrit.ics.uci.edu/1131
To unsubscrib
Jianfeng Jia has posted comments on this change.
Change subject: Adding a new 101 tutorial for SQL++.
..
Patch Set 1:
(7 comments)
Looks good to me.
All comments are more about the SQL++ syntax itself than the contents. Some
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by
..
Patch Set 5: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1144
To unsubscribe,
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by
..
Patch Set 4: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1144
To unsubscribe,
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by
..
Patch Set 3:
(1 comment)
https://asterix-gerrit.ics.uci.edu/#/c/1144/3/hyracks-fullstack/hyracks/hyra
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1628: Fixed an issue in External Hash Group by
..
Patch Set 1: Code-Review+2
--
To view, visit https://asterix-gerrit.ics.uci.edu/1144
To unsubscribe,
Jianfeng Jia has posted comments on this change.
Change subject: ASTERIXDB-1566,ASTERIXDB-1628: Fixed External Hash Group By to
conform to the memory budget
..
Patch Set 9:
@Taewoo, IMHO, the
`The number of partitions for the
Jianfeng Jia has posted comments on this change.
Change subject: Not suppose to MERGE, for discussion purpose The following
commits from your working branch will be included:
..
Patch Set 1:
@Yingyi, I think there maybe the is
Jianfeng Jia has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1135
Change subject: Not suppose to MERGE, for discussion purpose The following
commits from your working branch will be included:
..
Not s
Jianfeng Jia has posted comments on this change.
Change subject: Fixes related to ASTERIXDB-1534
..
Patch Set 2: Code-Review+1
--
To view, visit https://asterix-gerrit.ics.uci.edu/
To unsubscribe, visit https://asterix-ger
1 - 100 of 125 matches
Mail list logo